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.

Sign up for GitHub

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

Jump to bottom

test: network filter selector sandbox (DO NOT MERGE) #12251

Closed
wants to merge 163 commits into from
Closed

test: network filter selector sandbox (DO NOT MERGE) #12251

wants to merge 163 commits into from

Conversation

vinnyhoward
Copy link
Contributor

Description

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

gambinish and others added 30 commits September 26, 2024 17:09
@gambinish
feat: Add sortAssets utility function
0248c74
@gambinish
chore: Breakup monolithic Tokens/index.tsx into smaller components
6a2e8c7
@gambinish
fix: Correctly handle null check for buyable footer button
7a78717
@gambinish
fix: Tweak boolean logic for rendering footer components
57295ee
@gambinish
chore: Update snapshot
d00354f
@gambinish
chore: Type out tokens as TokenI[]
be87f81
@gambinish
chore: Remove unecessary comments around import statements in Networt…
68c7b2e 
…h component
@gambinish
Merge branch 'main' into chore/componetize-tokens-screen
a02e139
@gambinish
Merge branch 'main' into chore/componetize-tokens-screen
de5bbed
@gambinish
Merge branch 'main' into chore/componetize-tokens-screen
0fd9ad5
@gambinish
chore: Add unit tests to utility file, provide exports from index file
032618c
@gambinish
chore: Rename handleBalance to me more descriptive of what its logic …
a58f1ba 
…is handling
@gambinish
chore: rename Networth to PortfolioBalance for clarity
1dd31e3
@gambinish
chore: Cleanup styles extra whitespace
16ded6a
@gambinish
Merge branch 'main' into chore/componetize-tokens-screen
52d3d85
@gambinish
fix: Update snapshot tests
dd3ffdf
@gambinish
Merge branch 'chore/componetize-tokens-screen' of github.com:MetaMask…
da640bc 
…/metamask-mobile into chore/componetize-tokens-screen
@gambinish
fix: Remove unused component
ebcbcf6
@gambinish
Merge branch 'main' into chore/componetize-tokens-screen
31fad19
@gambinish
chore: Move PotfolioBalance above TabView
4a8d927
@gambinish
fix: Wrap PortfolioBalance and ScrollableView in fragment for token r…
a7b4c17 
…endering
@gambinish
fix: Padding tweak below tab bar for consistency of token list
658fcf6
@gambinish
fix: Reorganize test suite for PortfolioBalance
18035d4
@gambinish
fix: Revert changes to wallet test
c570c2c
@gambinish
fix: Update fiatBalance text style to DisplayMD
a97128b
@gambinish
chore: Patch PreferencesController with tokenSortConfig
8036a0a
@gambinish
chore: Patch preferences controller with type
1f0ac51
@gambinish
feat: Build token sorting UI, patch PreferencesController
34d987c
@gambinish
chore: Merge main, address merge conflicts
d57558b
@gambinish
fix: ListHeaderComponent not needed in FlatList anymore
c5c954c
bergeron and others added 20 commits November 7, 2024 14:35
@bergeron
add unit test
6028c8e
@bergeron
Merge branch 'main' into multichain-polling
bb25768
@vinnyhoward
fix: renaming test file to match component file
a06c896
@vinnyhoward
fix: revert change - wrong branch
82f69bd
@bergeron
lint
fedfb2b
@bergeron
Merge branch 'main' into multichain-polling
a6c04e5
@bergeron
remove things other than the hook
fbdadd9
@bergeron
improve unit test
8883861
@bergeron
format
22f34a4
@gambinish
fix: Remove unused font style
ab20b3e
@gambinish
fix: Use true for feature flag
fd28086
@vinnyhoward
Merge branch 'feat/mmassets-431_network-filter-ui' of github.com:Meta…
94e9bfe 
…Mask/metamask-mobile into feat-431-network-filter-selector
@gambinish
fix: Unblock podfile during CI
db79b71
@vinnyhoward
Merge branch 'feat/mmassets-431_network-filter-ui' of github.com:Meta…
1297288 
…Mask/metamask-mobile into feat-431-network-filter-selector
@vinnyhoward
feat: partial update to token list related logic
7ddba48
@vinnyhoward
Merge branch 'feat-431-network-filter-selector' of github.com:MetaMas…
ae232b1 
…k/metamask-mobile into feat-431-network-filter-selector
@vinnyhoward
test: update comments
2a71002
@vinnyhoward
stupid
c7c85a8
@vinnyhoward
Merge branch 'multichain-polling' of github.com:MetaMask/metamask-mob…
d6b63fd 
…ile into feat-431-network-filter-selector-sandbox
@vinnyhoward
testing
3559af1
@vinnyhoward vinnyhoward added DO-NOT-MERGE Pull requests that should not be merged No QA Needed Apply this label when your PR does not need any QA effort. No E2E Smoke Needed If the PR does not need E2E smoke test run labels Nov 9, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 9, 2024

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@gambinish gambinish changed the base branch from main to feat/mmassets-431_network-filter-ui November 9, 2024 01:14
@vinnyhoward vinnyhoward changed the title feat: network filter selector sandbox (DO NOT MERGE) test: network filter selector sandbox (DO NOT MERGE) Nov 9, 2024
@gambinish gambinish changed the base branch from feat/mmassets-431_network-filter-ui to main November 9, 2024 01:18
@github-actions github-actions bot locked and limited conversation to collaborators Nov 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
DO-NOT-MERGE Pull requests that should not be merged No E2E Smoke Needed If the PR does not need E2E smoke test run No QA Needed Apply this label when your PR does not need any QA effort. team-wallet-ux
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@vinnyhoward @metamaskbot @gambinish @salimtb @cryptodev-2s @sahar-fehri @bergeron