-
Notifications
You must be signed in to change notification settings - Fork 10
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
Ms2/modeler share modal #193
Conversation
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.
Looks good overall. Please fix the last issue and merge the main branch into this one
src/management-system-v2/components/modeler-share-modal-option-public-link.tsx
Outdated
Show resolved
Hide resolved
src/management-system-v2/components/modeler-share-modal-option-public-link.tsx
Outdated
Show resolved
Hide resolved
src/management-system-v2/components/modeler-share-modal-option-embed-in-web.tsx
Outdated
Show resolved
Hide resolved
src/management-system-v2/components/modeler-share-modal-option-embed-in-web.tsx
Outdated
Show resolved
Hide resolved
src/management-system-v2/components/modeler-share-modal-option-embed-in-web.tsx
Outdated
Show resolved
Hide resolved
src/management-system-v2/components/modeler-share-modal-option-public-link.tsx
Outdated
Show resolved
Hide resolved
src/management-system-v2/app/api/share/generate-share-token/route.ts
Outdated
Show resolved
Hide resolved
src/management-system-v2/app/api/share/generate-share-token/route.ts
Outdated
Show resolved
Hide resolved
src/management-system-v2/app/api/share/validate-share-token/route.ts
Outdated
Show resolved
Hide resolved
src/management-system-v2/components/modeler-share-modal-option-embed-in-web.tsx
Outdated
Show resolved
Hide resolved
src/management-system-v2/components/modeler-share-modal-option-public-link.tsx
Outdated
Show resolved
Hide resolved
src/management-system-v2/components/modeler-share-modal-option-public-link.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.
Selecting Embed in Website
or Share Process with Public Link
result in an error saying secretOrPrivateKey must have a value
The environment variable JWT_KEY= needs to be defined in .env.local file. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
… the platform in the current context, possibly because the user denied permission
… the platform in the current context, possibly because the user denied permission
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.
Looks overall good to me. There are smaller things but nothing that blocks merging in my eyes.
✅ Successfully created Preview Deployment. |
I have created the outline of the share dialog, tried to make it responsive as well.
Need to work on actual functionality of share public link and embed in website
Copy to Diagram Clipboard, Copy BPMN to Clipboard, Export as file work and leverage the functions implemented by Janis.