Skip to content

feat: Cell manage - #2859

Merged
Keith-CY merged 12 commits into
nervosnetwork:developfrom
yanguoyu:feat-cell-manage
Nov 24, 2023
Merged

feat: Cell manage#2859
Keith-CY merged 12 commits into
nervosnetwork:developfrom
yanguoyu:feat-cell-manage

Conversation

@yanguoyu

@yanguoyu yanguoyu commented Oct 7, 2023

Copy link
Copy Markdown
Contributor

The feature refers to Magickbase/neuron-public-issues#218.

  1. Light theme
cell-manage.mov
  1. Dark theme
dark-cell-manage.mov

@silySuper

silySuper commented Oct 12, 2023

Copy link
Copy Markdown

/package
Packaging for test is done in 6492616447. @silySuper

@silySuper

Copy link
Copy Markdown

@silySuper

silySuper commented Oct 16, 2023

Copy link
Copy Markdown

/package
Packaging for test is done in 6527989797. @silySuper

Comment thread packages/neuron-wallet/src/services/sudt-token-info.ts Outdated
Comment thread packages/neuron-wallet/src/services/cells.ts Outdated
Comment thread packages/neuron-wallet/src/services/cells.ts Outdated
@silySuper

silySuper commented Oct 17, 2023

Copy link
Copy Markdown

/package
Packaging for test is done in 6541661994. @silySuper

@yanguoyu
yanguoyu force-pushed the feat-cell-manage branch 2 times, most recently from f4fa8b6 to 6cf1cd6 Compare October 19, 2023 08:46
@silySuper

silySuper commented Oct 20, 2023

Copy link
Copy Markdown

/package
Packaging for test is done in 6585093792. @silySuper

@silySuper

silySuper commented Nov 15, 2023

Copy link
Copy Markdown

/package
Packaging for test is done in 6872605463. @silySuper

@silySuper

silySuper commented Nov 15, 2023

Copy link
Copy Markdown

/package
Packaging for test is done in 6874192894. @silySuper

@yanguoyu

yanguoyu commented Nov 19, 2023

Copy link
Copy Markdown
Contributor Author

/package
Packaging for test is done in 6919908812. @yanguoyu

Comment thread packages/neuron-wallet/src/controllers/cell-manage.ts
Comment thread packages/neuron-wallet/src/database/chain/entities/cell-local-info.ts Outdated
Comment thread packages/neuron-wallet/src/controllers/cell-manage.ts Outdated
@yanguoyu
yanguoyu requested a review from Keith-CY November 22, 2023 01:55
@Keith-CY

Copy link
Copy Markdown
Contributor

CI failed

@silySuper

silySuper commented Nov 22, 2023

Copy link
Copy Markdown

/package

Packaging for test is done in 6954535233. @silySuper

@yanguoyu

yanguoyu commented Nov 22, 2023

Copy link
Copy Markdown
Contributor Author

/package
Packaging for test is done in 6956454154. @yanguoyu

@yanguoyu

yanguoyu commented Nov 22, 2023

Copy link
Copy Markdown
Contributor Author

/package
Packaging for test is done in 6956529549. @yanguoyu

@yanguoyu

yanguoyu commented Nov 23, 2023

Copy link
Copy Markdown
Contributor Author

/package
Packaging for test is done in 6965061938. @yanguoyu

@silySuper

Copy link
Copy Markdown

verified.

@Keith-CY
Keith-CY added this pull request to the merge queue Nov 24, 2023
Merged via the queue into nervosnetwork:develop with commit 6dc2563 Nov 24, 2023
yanguoyu added a commit to yanguoyu/neuron that referenced this pull request Mar 14, 2024
* feat: Cell manage

* fix: Fix bugs of issue 298(1, 2, 3, 4, 5, 6)

* fix: Disable lock or unlock when all of the selected status is same.

Others optimize
1. rename `type` to `Category`
2. delete `walletId` in the `getSudtTokenInfo` function.

* fix: Add left join to add assetAccounts field.

* fix: Show the tips for locked cells.

* fix: Send max for consume appointed cells.

* fix: Show claim in Customized Assets if the lock time is reached.

* fix: Use filter option to get live and sent cell.

* fix: fix conversation

1. Better name cellManagement.
2. Move transformer to CellLocalInfo entity.
3. Use `UpdateCellLocalInfo` to replace multi params.

* fix: Fix test case

* fix: Adapt Spore for cell management.

* fix: Hide the balance when use consume cell
@Keith-CY Keith-CY mentioned this pull request Mar 14, 2024
@yanguoyu
yanguoyu deleted the feat-cell-manage branch May 14, 2024 03:03
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.

4 participants