Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Remove unsused state #2361

Merged
merged 3 commits into from
Feb 1, 2022
Merged

Remove unsused state #2361

merged 3 commits into from
Feb 1, 2022

Conversation

anxolin
Copy link
Contributor

@anxolin anxolin commented Jan 31, 2022

Summary

I noticed we use some state and reducers we don't need.

This PR tries to remove it. Ideally the app should work as before

To Test

  1. Test swap and claiming, make sure nothing is broken

@github-actions
Copy link
Contributor

  • 🔭 GP Swap: CoW Protocol v2 Swap UI

alfetopito
alfetopito approved these changes Jan 31, 2022
View reviewed changes
Copy link
Contributor

@alfetopito alfetopito left a comment

Choose a reason for hiding this comment

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

Looks alright

@anxolin anxolin merged commit 4955acc into develop Feb 1, 2022
@fairlighteth fairlighteth mentioned this pull request Feb 1, 2022
Merged
@alfetopito alfetopito deleted the remove-unused-state branch February 2, 2022 21:08
ramirotw pushed a commit that referenced this pull request Feb 18, 2022
@zzmp
fix: avoid replaceAll (#2361)
308f7d5
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alfetopito alfetopito alfetopito approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@anxolin @alfetopito @lint-action