Skip to content
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

249 new page certify #258

Open
wants to merge 36 commits into
base: main
Choose a base branch
from
Open

249 new page certify #258

wants to merge 36 commits into from

Conversation

ninokeldishvili
Copy link
Collaborator

@ninokeldishvili ninokeldishvili commented Aug 17, 2023

@ninokeldishvili ninokeldishvili linked an issue Aug 17, 2023 that may be closed by this pull request
@ninokeldishvili ninokeldishvili marked this pull request as ready for review August 22, 2023 13:49
@ichniten
Copy link
Collaborator

Screenshot 2023-08-22 at 18 46 14

Audit history to have a grey 'certify' button at the base of it that connects to the 'certify' page

@ichniten
Copy link
Collaborator

ichniten commented Aug 25, 2023

Screenshot 2023-08-25 at 12 26 34

the 'cerify until' that's not recent to be chaged to grey in colour, not red, and the active certified to be green/ red depending on the time

Screenshot 2023-08-25 at 12 35 25

On certify page: 'Asset class' to be titled 'Audit type', and 'Asset info.' to be titled 'Audit info.'

Screenshot 2023-08-25 at 12 43 38

the audit report has been finalised, it can be found here

@ichniten
Copy link
Collaborator

ichniten commented Aug 25, 2023

Screenshot 2023-08-25 at 17 45 55

Checkbox added to the design. The checkbox being not checked by default

@ichniten
Copy link
Collaborator

it seems to work fine, a red warning message needs to be aded to some pages to show the 'system is frozen until certified' though

@ichniten
Copy link
Collaborator

Screenshot 2023-08-29 at 16 53 12

All pages circled in red need a warning on them if the system has been frozen

@ichniten
Copy link
Collaborator

Screenshot 2023-08-30 at 12 28 58

Warning, this token is frozen. {{active certification link}}

The line above to be added to the folllowing pages at the top of screen (on row underneath the breadcrumbs). 'Audit history' page has in next to the certify button.

Mint
Redeem
New asset class
Asset class list
Asset class register
Asset history
SFT roles
Members
Certify
Audit report
Confiscate

@ichniten
Copy link
Collaborator

This warning label also be on the pages: 'Asset information' and 'new revision'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

new page - certify
2 participants