Skip to content

Conversation

ekzyis
Copy link
Member

@ekzyis ekzyis commented Apr 15, 2025

Description

We don't use these wallet resolver and arguments anymore.

Removing them also means less stuff for me to migrate in #2092.

Checklist

Are your changes backwards compatible? Please answer below:

yes since the resolvers weren't used

On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:

8. I noticed that the resolvers are unused because I found nothing referencing the fragments. Also tried to search manually for some references but none showed up.

For frontend changes: Tested on mobile, light and dark mode? Please answer below:

n/a

Did you introduce any new environment variables? If so, call them out explicitly here:

no

@ekzyis ekzyis added the wallets label Apr 15, 2025
@ekzyis ekzyis changed the title Remove unused resolvers Remove unused wallet resolvers Apr 15, 2025
@huumn huumn merged commit 9b77cf0 into master Apr 15, 2025
6 checks passed
@huumn huumn deleted the remove-unused-resolvers branch April 15, 2025 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants