[BUILD-748] feat: Add function to send the daily summary review data after the sync - Githubissues

AnkiHubSoftware / ankihub_addon

3 stars 0 forks source link

[BUILD-748] feat: Add function to send the daily summary review data after the sync #1025

Closed pedroven closed 1 month ago

pedroven commented 1 month ago

Related issues

BUILD-748

Proposed changes

This PR makes these changes:

Add the function send_daily_review_summaries which will send to the AnkiHub server the compiled data of the daily reviews
Call this function in the background after the user sync
Update the feature_flags.py module and its use to avoid the need to update the dataclass if a new feature flag is created on the server

How to reproduce

Run the instance of the ankihub server and make sure that the feature flag daily_card_review_summary is enabled
Open a deck and review a card
Open the anki database and update the ID of the newly created revlog row to the value of a timestamp of yesterday's date
Click on the "Sync with AnkiHub" option on the Ankihub menu
Go to a shell instance of the Ankihub server and check if a register of the DailyCardReviewSummary was created successfully

pedroven commented 1 month ago

@RisingOrange I don't know if you noticed but seems like some tests are flaky, sometimes the CI pipeline fails, and sometimes not.

RisingOrange commented 1 month ago

@RisingOrange I don't know if you noticed but seems like some tests are flaky, sometimes the CI pipeline fails, and sometimes not.

Yes, this is indeed an issue. I haven't got around to fixing it yet