Anki-Android
Anki-Android copied to clipboard
Factorize deck pref
I was looking at #11417 and this led me to find far more duplication than I expected. If it's too hard to review all at once, I can split. The last commit is the hardest one. The first few commits, by removing some nullability thanks to late init, are really nice by themselves. Then many commits are factorizing copy-pasted code and can be considered by themselves (but would conflict with the first ones)
Converting this to draft since it's not really an actionable PR from a review perspective, with the work split out into multiple little PRs
Hello 👋, this PR has been opened for more than 2 months with no activity on it. If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing! You have 7 days until this gets closed automatically
Conflict resolved
@Arthur-Milchior this has some sort of compile error :thinking: ?
@Arthur-Milchior squash or rebase?
@david-allison Currently neither. You did the two approval. We should maybe wait for a review by someone else. Later, rebase
@lukstbit I had no knowledge about it, I admit that enough people seems to follow the new back-end that I don’t feel like I had to go read it
The new backend code will have a large impact on the future of AnkiDroid development, so as someone who reviews other people's PRs, I'd recommend getting up to speed with it.
Any advice where to start reading about it?
In this case it requires keeping track of active/merged PRs - https://github.com/ankidroid/Anki-Android/pull/12175 is the one that will introduce the new decks screen.
Related info:
- https://github.com/ankidroid/Anki-Android/issues/11882
- https://github.com/ankidroid/Anki-Android-Backend/blob/main/docs/OVERVIEW.md
Hi there @Arthur-Milchior! This is the OpenCollective Notice for PRs merged from 2022-09-01 through 2022-09-30
If you are interested in compensation for this work, the process with details is here:
https://github.com/ankidroid/Anki-Android/wiki/OpenCollective-Payment-Process#how-to-get-paid
We only post one comment per person per month to avoid spamming you, regardless of the number of PRs merged, but this note applies to all PRs merged for this month
Please note that GSoC contributions are okay for this process. Our philosophy is that our users have donated to AnkiDroid for all contributions. The only PRs that will not go through the OpenCollective process are ones directly related to am accepted GSoC project from a selected participant, since those receive a stipend from GSoC itself.
Please understand that our monthly budget is never guaranteed to cover all claims - the cap on payments-per-person may be lower, but we try to make our process as fair and transparent as possible, we just need your understanding.
Thanks!