Skip to content

Add view after verification#4195

Open
agnieszkajarosikloj wants to merge 19 commits into
feature/4153-view-after-verification-submissionfrom
feature/4190-view-after-verification
Open

Add view after verification#4195
agnieszkajarosikloj wants to merge 19 commits into
feature/4153-view-after-verification-submissionfrom
feature/4190-view-after-verification

Conversation

@agnieszkajarosikloj

Copy link
Copy Markdown

Description

This PR adds view for not nominated users (banner is not visible) or users that have successfully validated (banner is not visible and label "Verified" is displayed).
Checking if the user has permission to create a payment motion has also been added (it is only allowed for nominated users or for the owner). Checking is a mock, needs to be replaced with backend data.

Testing

  • To test this view go to /incorporation/create route and fill in the form.
  • If you don't add yourself as a protector, you will not be able to create motion payment.

Design

*Figma Link: Standard view user users not nominated

Screenshot 2023-02-09 at 15 39 04

Resolves #4190

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@arrenv arrenv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The state looks good, the details in your description regarding required permissions to create a motion are not relevant here though.

@agnieszkajarosikloj agnieszkajarosikloj force-pushed the feature/4153-view-after-verification-submission branch from ce0508e to ec59e27 Compare March 13, 2023 14:10
@agnieszkajarosikloj agnieszkajarosikloj force-pushed the feature/4190-view-after-verification branch from 18eb06e to ccf2345 Compare March 21, 2023 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants