-
Notifications
You must be signed in to change notification settings - Fork 19
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
Add manage application button #4182
base: feature/4126-sidebar-locked
Are you sure you want to change the base?
Add manage application button #4182
Conversation
|
30abc95
to
f162ee8
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.
Thanks @agnieszkajarosikloj. It really would be good to have these rebased on required branches so only actual changes are visible.
They don't seem to align exactly -
</span> | ||
)} | ||
</div> | ||
{hasPermissions && ( |
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.
Can we please add a comment here that this button needs to be wired up.
6b49a42
to
c96ee07
Compare
@arrenv I added changes. Can you have a look? |
Description
This PR adds "Manage application" button.
To test go to /incorporation/create route and fill in the form.
Logic for managing application will be added in Stage 2.
Figma Link - Manage application button
Resolves #4179