Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix note generator audio #1278

Merged
merged 2 commits into from
Jul 28, 2024
Merged

Conversation

Kuuuube
Copy link
Member

@Kuuuube Kuuuube commented Jul 28, 2024

Fixes a regression causing the anki note generator to fail downloading audio. This regression was introduced in #1152 and snuck in primarily through changes made in #1129 which did not cause any direct issue.

Also added a log for when audio fails to download. This never triggers when the user does not have the {audio} in their card template.

@Kuuuube Kuuuube added kind/bug The issue or PR is regarding a bug area/anki The issue or PR is related to Anki integration area/audio The issue or PR is related to audio labels Jul 28, 2024
@Kuuuube Kuuuube requested a review from a team as a code owner July 28, 2024 03:49
@Kuuuube Kuuuube added this pull request to the merge queue Jul 28, 2024
Merged via the queue into yomidevs:master with commit 2644f2f Jul 28, 2024
10 checks passed
@Kuuuube Kuuuube deleted the fix-note-gen-audio branch July 28, 2024 04:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/anki The issue or PR is related to Anki integration area/audio The issue or PR is related to audio kind/bug The issue or PR is regarding a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants