Tarek Mohamed Abdalla

Results 131 comments of Tarek Mohamed Abdalla

I have this same issue. ![Screen Shot 2022-05-26 at 6 26 30 PM](https://user-images.githubusercontent.com/6633545/170531541-c7a06926-8a60-4a93-978a-de3489a2a031.png)

> Do we need a new decorator for this, or is `@freezed` enough? > > I would assume that we don't need one. It could be inferred based on: >...

Thank you about the wonderful feedback. I really appreciate you taking the time to write it. I will try to implement some tests for the new ported code, will need...

I will not be able to work on this PR for the next few days, as I will be having exams this week. So I will try to work sparingly,...

@Arthur-Milchior I have cleaned up the miss, and have divided commits into multiple logical ones, with appropriate messages. Now there is no unrelated code formatting issues (lines, spaces). I have...

Thanks a lot @Arthur-Milchior for the feedback. I'm referring to it as "Individual **cards**" that the user have selected, maybe I'm using the expression wrong, if so should I change...

No issues will push a new PR with that. I'm also not a native speaker of English, so I would not know if this is 100% correct, but I feel...

I have renamed the PR and the previous commit messages. Will work on the cleaning PR and submit it

@Arthur-Milchior I have rebased this PR over #8196. I think it's ready for a review

@Arthur-Milchior can you have a look at this. This is a blocker to adding the CSV export. The libanki porting is merged, what is left is this refactoring