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

Connector server support to adding a note #5064

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

abaevbog
Copy link
Contributor

  • pass canUserAddNote flag in ping response to tell the connector that it should display a note field
  • when the connector sends a note, create a child note item if it does not yet exist and set its content. If the child item for user-added notes is already created, update it
  • if the connector sends an empty note, user-added child note item is deleted
  • record child notes added by the user in _userAddedNotes cache to distinguish between user added notes and
    notes added by the connector automatically
  • if the target library changes, child note items are moved to the new library as well. _userAddedNotes
    will be updated to reflect new itemIDs
  • if there are multiple toplevel items being added, the note is added to all of them

- pass canUserAddNote flag in ping response to tell
the connector that it should display a note field
- when the connector sends a note, create a child note item
if it does not yet exist and set its content. If the
child item for user-added notes is already created,
update it
- if the connector sends an empty note, user-added child
note item is deleted
- record child notes added by the user in _userAddedNotes
cache to distinguish between user added notes and
notes added by the connector automatically
- if the target library changes, child note items are
moved to the new library as well. _userAddedNotes
will be updated to reflect new itemIDs
- if there are multiple toplevel items being added,
the note is added to all of them
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant