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

audits info and form #1598

Merged
merged 8 commits into from
Feb 19, 2024
Merged

audits info and form #1598

merged 8 commits into from
Feb 19, 2024

Conversation

isstuev
Copy link
Collaborator

@isstuev isstuev commented Feb 11, 2024

Description and Related Issue(s)

resolves #1563

Proposed Changes

adds NEXT_PUBLIC_HAS_CONTRACT_AUDIT_REPORTS variable to enable feature

Checklist for PR author

  • I have tested these changes locally.
  • I added tests to cover any new functionality, following this guide
  • Whenever I fix a bug, I include a regression test to ensure that the bug does not reappear silently.
  • If I have added, changed, renamed, or removed an environment variable, I have updated the list of environment variables in the documentation and made the necessary changes to the validator script according to the guide

Copy link
Collaborator

@tom2drum tom2drum left a comment

Choose a reason for hiding this comment

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

I think it is better to disable autosuggest for all form fields except email and username

image

ui/address/contract/ContractSecurityAudits.tsx Outdated Show resolved Hide resolved
ui/address/contract/ContractCode.pw.tsx Outdated Show resolved Hide resolved
@isstuev isstuev merged commit 5c2d06f into main Feb 19, 2024
7 checks passed
@isstuev isstuev deleted the fe-1563 branch February 19, 2024 13:30
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.

Contract audit URL
3 participants