-
Notifications
You must be signed in to change notification settings - Fork 52
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
Issue 2096/error handling for notes upload #2204
Issue 2096/error handling for notes upload #2204
Conversation
265d9bc
to
a35e978
Compare
a35e978
to
9f00846
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome work! I've tried to preview build to confirm that it works, and I've reviewed the code which looks mostly very good. I don't love how many components are involved though, so I'm wondering if it would make sense to keep the state within ZUITimeline
. But I can totally be convinced otherwise!
I would like to change the text of the error message though. 👇
src/pages/organize/[orgId]/journeys/[journeyId]/[instanceId]/index.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I appreciate it! Great work on this. 👏
Description
This PR makes it so you get an error message when trying to upload a too long journey note, instead of just annihilating your hard-written text.
Screenshots
Changes
Related issues
Resolves #2096