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 tags not displaying for duplicate cards #819

Merged
merged 1 commit into from
Apr 15, 2024

Conversation

Kuuuube
Copy link
Member

@Kuuuube Kuuuube commented Apr 15, 2024

Fixes #817

cannotAdd is no longer a possible differentiator for if a card is a dupe and cannot be used to check whether notesInfo needs to be pulled.

@Kuuuube Kuuuube requested a review from a team as a code owner April 15, 2024 02:06
Copy link

✔️ No visual differences introduced by this PR.

View Playwright Report (note: open the "playwright-report" artifact)

@StefanVukovic99 StefanVukovic99 added the kind/bug The issue or PR is regarding a bug label Apr 15, 2024
Copy link
Collaborator

@jamesmaa jamesmaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Just some questions for my own edification

ext/js/background/backend.js Show resolved Hide resolved
ext/js/background/backend.js Show resolved Hide resolved
@jamesmaa jamesmaa enabled auto-merge April 15, 2024 17:22
@jamesmaa jamesmaa added this pull request to the merge queue Apr 15, 2024
Merged via the queue into yomidevs:master with commit f5a6b25 Apr 15, 2024
10 checks passed
@Kuuuube Kuuuube added the area/anki The issue or PR is related to Anki integration label Apr 16, 2024
github-merge-queue bot pushed a commit that referenced this pull request Apr 16, 2024
* added Old Irish (SGA)

* Merge Yomitan Updates (#1)

* fix (#811)

* Add scanOnTouchTap and improve touch scanning defaults (#791)

* Add scanOnTouchTap

* Update version to 30

* Cleanup if statement

* log anki error when hiding button (#821)

* Fix noteInfos not getting assigned (#819)

* improve term sorting (#806)

* improve term sorting

* edge case

* fix: add missing handlebar entry (#823)

* fix: add missing handlebar entry

* fix: add new handlebar to tests

* Revert to using canAddNotes (#827)

Fix #818

* Remove unused canAdd (#824)

Co-authored-by: James Maa <[email protected]>

* Fix duplicate check not working across note types (#830)

* Fix duplicate check not working across note types

* Add invalidNoteId

---------

Co-authored-by: James Maa <[email protected]>

---------

Co-authored-by: StefanVukovic99 <[email protected]>
Co-authored-by: Kuuuube <[email protected]>
Co-authored-by: m-edlund <[email protected]>
Co-authored-by: Eloy Robillard <[email protected]>
Co-authored-by: James Maa <[email protected]>
Co-authored-by: James Maa <[email protected]>

* Revert "Merge Yomitan Updates (#1)"

This reverts commit 748dc22.

* Fix Static Analysis Error

* Fix eslint erro

---------

Co-authored-by: martholomew <[email protected]>
Co-authored-by: StefanVukovic99 <[email protected]>
Co-authored-by: Kuuuube <[email protected]>
Co-authored-by: m-edlund <[email protected]>
Co-authored-by: Eloy Robillard <[email protected]>
Co-authored-by: James Maa <[email protected]>
Co-authored-by: James Maa <[email protected]>
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 kind/bug The issue or PR is regarding a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show card tags feature not working
3 participants