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

Staging #36

Merged
merged 9 commits into from
Oct 14, 2021
Merged

Staging #36

merged 9 commits into from
Oct 14, 2021

Conversation

travis
Copy link
Contributor

@travis travis commented Oct 14, 2021

No description provided.

Follow the pattern we were using from plate of having a button in the toolbar that inserts an image. Customize that button because it was not stopping event propagation which would auto-close the modal dialog.

Factor out a "Modal" component.

Also get rid of legacy button classes.
this is the Plate version of the same function so should be a bit better
This thing is a bit of a beast.

In order to make a note private at the moment we move it into a different directory inside either
the public or private directories in the user's POD. We do this because we don't want to muck around with
ACLs quite yet, but this design decision should be revisited soon.

For the moment, this means that we need to do several operations when we change privacy:

1) create the new note resource
2) update both the public and private index and finally
3) delete the old note resource

This is why we have so many hooks below, and two separate functions for making a note public or private.
use useStoreEditorRef instead of useSlateStatic
Get the privacy toggle working again
 Don't show editing controls on other peoples' notes
@vercel
Copy link

vercel bot commented Oct 14, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/mysilio/garden/GgzWYXRx4nsAbXAiXRFDeAk6Epaj
✅ Preview: https://staging.mysilio.garden

@travis travis merged commit bfe8777 into main Oct 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant