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

Check claims disconnected #2281

Merged
merged 2 commits into from Jan 25, 2022
Merged

Check claims disconnected #2281

merged 2 commits into from Jan 25, 2022

Conversation

alfetopito
Copy link
Contributor

Summary

Allow checking for claims while disconnected

Screen Shot 2022-01-24 at 16 21 18

To Test

  1. Open the claim page and DO NOT connect your wallet
  2. Check for accounts which should have claims in mainnet
  • The claims should be visible

  • The button should say Connect wallet

  • Everything else should work as usual

    Background

Optional: Give background information for changes you've made, that might be difficult to explain via comments

@alfetopito alfetopito self-assigned this Jan 25, 2022
@alfetopito alfetopito requested review from a team January 25, 2022 00:25
@github-actions
Copy link
Contributor

  • 🔭 GP Swap: CoW Protocol v2 Swap UI

Copy link

@elena-zh elena-zh left a comment

Choose a reason for hiding this comment

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

Great job!

Maybe as a future enhancement, I'd like to propose to add 'Connect to a wallet' message or change 'Check claimable vCOW' button to 'Connect to a wallet' for the next screen
image

WDYT?

Copy link
Contributor

@anxolin anxolin left a comment

Choose a reason for hiding this comment

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

Works great!

@anxolin anxolin merged commit 19adb51 into develop Jan 25, 2022
maria-vslvn pushed a commit that referenced this pull request Jan 25, 2022
* Add Pin to Crust workflow

* Upgrade crust pinner adapt Crust Mainnet
@alfetopito alfetopito deleted the check-claims-disconnected branch January 25, 2022 15:52
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.

None yet

4 participants