[GSoC'24] Analytical widget provider class for Deck Picker Widget .
Purpose / Description
Adding Up Analytical widget provider class for Deck Picker Widget .
How Has This Been Tested?
[GSoC'24] Implementation of Deck Picker Widget . #16450
Checklist
Please, go through these checks before submitting the PR.
- [x] You have a descriptive commit message with a short title (first line, max 50 chars).
- [x] You have commented your code, particularly in hard-to-understand areas
- [x] You have performed a self-review of your own code
- [x] UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
- [ ] UI Changes: You have tested your change using the Google Accessibility Scanner
You should use this with our current classes
You should use this with our current classes
Sorry, didn't understand what you meant by that
Two widget classes appear to be using this functionality and have duplicated code. You're planning on adding a third which also uses this functionality, so the concern was extracted to a class
You should add the class which you've created into the inheritance hierarchy for the two other classes to reduce complexity
I don't understand you "how has this been tested" section. Can you please explain how the link refers to tests ?
Pinged Arthur for re-review
Still has conflicts
- please squash merge
@Arthur-Milchior
Why is this no longer applied to
AnkiDroidWidgetSmall?
for some reason widget_eta and widget_due are not rendering after these changes , was investigating that onl;y
You should be communicating that this is the case
Hi there @xenonnn4w! This is the OpenCollective Notice for PRs merged from 2024-07-01 through 2024-07-31
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
[!IMPORTANT] PLEASE NOTE: The process was updated in August 2024. Re-read the Payment Process page if you have not already.
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 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!