-
Notifications
You must be signed in to change notification settings - Fork 181
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
Signing templates #618
base: main
Are you sure you want to change the base?
Signing templates #618
Conversation
Feat/template
@rajdip-b okey here is the update:
|
hi @rajdip-b was the update from Niko helpful? |
- fixed lots of conflicts - added lots of @todo to places where the actions/loaders need to be updated to new approach - added lots of @ts-expect-error for places where updates are needed to follow new method
@rajdip-b Updated the PR with main. |
… signing-templates
… signing-templates
Signing templates - rajdip
Points that still need to be worked on:
|
- Added migration to enable RLS for TemplateFile table - Updated actions that werent handling async correctly - Removed makeActive and makeInactive for template and replaced them with toggleTemplateActiveState - Completely updated template actions and forms - Updated some check in/check out labels - Fixed texts for notifications for changing default and activate/deactivate template - Updated/fixed disabled states of all template actions
fixed bugs with custody assignment
We use the
signing-templates
branch to keep track of everything related to the templates functionality.At the time of creating this PR the branch already has the functionality for creating the templates, however we have 2 remaining issues for it: