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

Disable claiming: BARN / PROD / ENS #2298

Merged
merged 2 commits into from
Jan 25, 2022
Merged

Disable claiming: BARN / PROD / ENS #2298

merged 2 commits into from
Jan 25, 2022

Conversation

anxolin
Copy link
Contributor

@anxolin anxolin commented Jan 25, 2022

Summary

This PR removes in production, barn and ENS the claiming so its hidden.

For people trying to use it ahead of time, note it's not possible 😆
basically because the functionality is hooked to FAKE tokens. The REAL token would be deploy by the DAO if they decide to deploy it.

Removes the button from the header:
Screenshot at Jan 25 16-22-18

Removes the balance from the Profile:
Screenshot at Jan 25 16-22-33

It evens remove the page:
Screenshot at Jan 25 16-22-06

To Test

  1. Test it locally, change the flag in the src/custom/pages/Claim/const.ts file
  2. Verify the app behaves as it shoud

@github-actions
Copy link
Contributor

  • 🔭 GP Swap: CoW Protocol v2 Swap UI

@anxolin anxolin marked this pull request as ready for review January 25, 2022 16:31
@anxolin anxolin changed the title Disable claiming Disable claiming: BARN / PROD / ENS Jan 25, 2022
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.

Code looks good, PR deploy failed

@anxolin
Copy link
Contributor Author

anxolin commented Jan 25, 2022

Code looks good, PR deploy failed

There was an error pushing...

@W3stside W3stside merged commit 435874c into release/1.10 Jan 25, 2022
@W3stside W3stside deleted the hide-claiming branch January 25, 2022 17:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants