feat: tag notes index #546

moughxyz · 2022-01-11T18:11:20Z

No description provided.

packages/snjs/lib/protocol/collection/tag_notes_index.ts

packages/snjs/lib/services/item_manager.ts

packages/snjs/lib/protocol/collection/tag_notes_index.ts

moughxyz requested a review from laurentsenta January 11, 2022 18:11

moughxyz marked this pull request as draft January 11, 2022 18:11

laurentsenta reviewed Jan 11, 2022

View reviewed changes

packages/snjs/lib/protocol/collection/tag_notes_index.ts Show resolved Hide resolved

laurentsenta reviewed Jan 11, 2022

View reviewed changes

packages/snjs/lib/services/item_manager.ts Outdated Show resolved Hide resolved

laurentsenta reviewed Jan 11, 2022

View reviewed changes

packages/snjs/lib/protocol/collection/tag_notes_index.ts Outdated Show resolved Hide resolved

moughxyz marked this pull request as ready for review January 11, 2022 18:33

moughxyz mentioned this pull request Jan 11, 2022

feat: use snjs tag notes index for note counts standardnotes/app#810

Merged

moughxyz and others added 9 commits January 17, 2022 10:24

feat: tag notes index

91a33ca

fix: remove redundant tag lookup

a5a10dc

fix: PR review

7780747

fix: find or create set

0990b2f

feat: note count change observer

0370d58

refactor: restructure the indexing internals

9349222

doc: add more intuitive e2e commands

7945488

refactor: allow countable over smarttags

2fda84d

fix: type import

a0cca74

laurentsenta force-pushed the feat/tag-notes-index branch from 8bdde6b to a0cca74 Compare January 17, 2022 09:47

laurentsenta approved these changes Jan 17, 2022

View reviewed changes

Merge branch 'master' into feat/tag-notes-index

d0f137f

moughxyz merged commit ab12281 into master Jan 17, 2022

moughxyz deleted the feat/tag-notes-index branch January 17, 2022 13:12

Provide feedback