Skip to content

feat: Go Plus Security plugin - #5873

Merged
guanbinrui merged 40 commits into
developfrom
feat/token-security-application
Mar 31, 2022
Merged

feat: Go Plus Security plugin#5873
guanbinrui merged 40 commits into
developfrom
feat/token-security-application

Conversation

@Lanttcat

@Lanttcat Lanttcat commented Mar 14, 2022

Copy link
Copy Markdown
Contributor

Description

Closes #MF-167

Type of change

  • Documentation
  • Code refactoring (Restructuring existing code w/o changing its observable behavior)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (a fix or feature that would make something no longer possible to do/require old user must upgrade their Mask Network to this new version)

Previews

Checklist

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
    • I have removed all in development console.logs
    • I have removed all commented code.
  • I have commented on my code, particularly in hard-to-understand areas.
  • I have read Internationalization Guide and moved text fields to the i18n JSON file.

If this PR depends on external APIs:

  • I have configured those APIs with CORS headers to let extension requests get passed.
    • chrome extension: chrome-extension://[id]
    • firefox extension: moz-extension://[id]
  • I have delegated all web requests to the background service via the internal RPC bridge.

@Lanttcat
Lanttcat marked this pull request as draft March 14, 2022 11:19
@Lanttcat Lanttcat changed the title Go+ Security plugin feat: Go Plus Security plugin Mar 14, 2022
@github-actions

github-actions Bot commented Mar 14, 2022

Copy link
Copy Markdown
Contributor

@github-actions
github-actions Bot temporarily deployed to pull request March 14, 2022 11:25 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request March 14, 2022 11:41 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request March 17, 2022 07:12 Inactive
@Lanttcat
Lanttcat force-pushed the feat/token-security-application branch from 98e6ec4 to 561d37b Compare March 17, 2022 08:21
@github-actions
github-actions Bot temporarily deployed to pull request March 17, 2022 08:27 Inactive
@Lanttcat
Lanttcat force-pushed the feat/token-security-application branch from 711f392 to e264c8d Compare March 21, 2022 03:48
@github-actions
github-actions Bot temporarily deployed to pull request March 21, 2022 09:59 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request March 21, 2022 10:25 Inactive
@Lanttcat
Lanttcat force-pushed the feat/token-security-application branch from 3ccb468 to 1b0a008 Compare March 24, 2022 14:42
@github-actions
github-actions Bot temporarily deployed to pull request March 24, 2022 14:47 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request March 25, 2022 09:38 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request March 25, 2022 09:46 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request March 25, 2022 10:44 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request March 28, 2022 03:41 Inactive
@Lanttcat
Lanttcat force-pushed the feat/token-security-application branch from ff61f4d to fdf16df Compare March 28, 2022 09:00
@github-actions
github-actions Bot temporarily deployed to pull request March 28, 2022 09:05 Inactive
Comment thread packages/mask/shared/index.ts Outdated
@orange187

Copy link
Copy Markdown

Checked

@Lanttcat
Lanttcat force-pushed the feat/token-security-application branch from e6afa87 to c712d59 Compare March 30, 2022 09:49
@github-actions
github-actions Bot temporarily deployed to pull request March 30, 2022 10:42 Inactive
@Lanttcat
Lanttcat marked this pull request as ready for review March 30, 2022 11:04
@github-actions
github-actions Bot temporarily deployed to pull request March 30, 2022 11:08 Inactive
Comment thread packages/plugins/GoPlusSecurity/src/SNSAdaptor/CheckSecurityDialog.tsx Outdated
Comment thread packages/plugins/GoPlusSecurity/src/SNSAdaptor/components/SearchBox.tsx Outdated
guanbinrui
guanbinrui previously approved these changes Mar 31, 2022
@github-actions
github-actions Bot temporarily deployed to pull request March 31, 2022 03:58 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request March 31, 2022 06:47 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request March 31, 2022 07:00 Inactive
@guanbinrui
guanbinrui merged commit c250fe6 into develop Mar 31, 2022
@guanbinrui
guanbinrui deleted the feat/token-security-application branch March 31, 2022 07:16
guanbinrui added a commit that referenced this pull request Mar 31, 2022
* feat: go plus security plugin setup

* feat: add get supported chain for go plus lab

* feat: plugin frame step 1

* feat: add risk rules

* feat: search box style

* feat: panel layout

* feat: panel style

* feat: update i18n

* feat: update rules

* style: improve

* style: interaction design improve

* feat: add default safa item card

* feat: all the field is optional

* feat: use go plus lab url proxy

* chore: lock file

* feat: improve

* fix: cspell

* feat: link to explorer

* chore: migrate mask message

* feat: configurable app board

* Revert "chore: migrate mask message"

This reverts commit 817e433.

* chore: resolve conflict

* chore: resolve conflict

* feat: open from application entry

* feat: dialog background

* feat: update base info

* feat: footer goto go plus website

* fix: order by risk

* style: security panel height

* refactor: create token panel component

* feat: use transition for token info collapse

* fix: should clear state when close dialog

* fix: remove debug code

* fix: rename network to chain

* fix: feedback

* style: dialog size

* refactor: convert chain's id type from string to chain id

* refactor: code review feedback

* fix: should can be use when not connect wallet

Co-authored-by: zhouhanseng <z308114274@gmail.com>
Co-authored-by: guanbinrui <52657989+guanbinrui@users.noreply.github.com>
Lanttcat added a commit that referenced this pull request Apr 6, 2022
* feat: go plus security plugin setup

* feat: add get supported chain for go plus lab

* feat: plugin frame step 1

* feat: add risk rules

* feat: search box style

* feat: panel layout

* feat: panel style

* feat: update i18n

* feat: update rules

* style: improve

* style: interaction design improve

* feat: add default safa item card

* feat: all the field is optional

* feat: use go plus lab url proxy

* chore: lock file

* feat: improve

* fix: cspell

* feat: link to explorer

* chore: migrate mask message

* feat: configurable app board

* Revert "chore: migrate mask message"

This reverts commit 817e433.

* chore: resolve conflict

* chore: resolve conflict

* feat: open from application entry

* feat: dialog background

* feat: update base info

* feat: footer goto go plus website

* fix: order by risk

* style: security panel height

* refactor: create token panel component

* feat: use transition for token info collapse

* fix: should clear state when close dialog

* fix: remove debug code

* fix: rename network to chain

* fix: feedback

* style: dialog size

* refactor: convert chain's id type from string to chain id

* refactor: code review feedback

* fix: should can be use when not connect wallet

Co-authored-by: zhouhanseng <z308114274@gmail.com>
Co-authored-by: guanbinrui <52657989+guanbinrui@users.noreply.github.com>
Lanttcat added a commit that referenced this pull request Apr 6, 2022
* feat: go plus security plugin setup

* feat: add get supported chain for go plus lab

* feat: plugin frame step 1

* feat: add risk rules

* feat: search box style

* feat: panel layout

* feat: panel style

* feat: update i18n

* feat: update rules

* style: improve

* style: interaction design improve

* feat: add default safa item card

* feat: all the field is optional

* feat: use go plus lab url proxy

* chore: lock file

* feat: improve

* fix: cspell

* feat: link to explorer

* chore: migrate mask message

* feat: configurable app board

* Revert "chore: migrate mask message"

This reverts commit 817e433.

* chore: resolve conflict

* chore: resolve conflict

* feat: open from application entry

* feat: dialog background

* feat: update base info

* feat: footer goto go plus website

* fix: order by risk

* style: security panel height

* refactor: create token panel component

* feat: use transition for token info collapse

* fix: should clear state when close dialog

* fix: remove debug code

* fix: rename network to chain

* fix: feedback

* style: dialog size

* refactor: convert chain's id type from string to chain id

* refactor: code review feedback

* fix: should can be use when not connect wallet

Co-authored-by: zhouhanseng <z308114274@gmail.com>
Co-authored-by: guanbinrui <52657989+guanbinrui@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants