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

Set initial claim invest input amount to an empty string #2280

Merged
merged 1 commit into from
Jan 25, 2022

Conversation

alfetopito
Copy link
Contributor

Summary

Suuuper big issue

Changing from:
Screen Shot 2022-01-24 at 15 41 54

To:
Screen Shot 2022-01-24 at 15 42 08

To Test

  1. Go to a paid claim
  • Input amount should be empty (placeholder is a 0)

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

  • 🔭 GP Swap: CoW Protocol v2 Swap UI

@alfetopito alfetopito merged commit 49425bf into develop Jan 25, 2022
@alfetopito alfetopito deleted the initial-claim-investment-input branch January 25, 2022 01:01
@elena-zh
Copy link

Changed LGTM!
However, I'd rather not to show validation message for the input when it is in a 'default' state with a placeholder in it. I reported it in #2285

maria-vslvn pushed a commit that referenced this pull request Jan 25, 2022
Bumps [tar](https://github.com/npm/node-tar) from 6.1.4 to 6.1.11.
- [Release notes](https://github.com/npm/node-tar/releases)
- [Changelog](https://github.com/npm/node-tar/blob/main/CHANGELOG.md)
- [Commits](isaacs/node-tar@v6.1.4...v6.1.11)

---
updated-dependencies:
- dependency-name: tar
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

4 participants