Skip to content

Feature/currency select disabled tooltip#559

Merged
RafaelVidaurre merged 9 commits intodevelopfrom
feature/currency-select-disabled-tooltip
Apr 2, 2019
Merged

Feature/currency select disabled tooltip#559
RafaelVidaurre merged 9 commits intodevelopfrom
feature/currency-select-disabled-tooltip

Conversation

@Grsmto
Copy link
Copy Markdown
Contributor

@Grsmto Grsmto commented Mar 29, 2019

Please make sure the following boxes are checked before submitting your Pull Request:

  • I've added this PR's link to its Asana task(s)
  • If this PR adds new code that is not going to change in the near future, it includes unit tests to cover it.

Copy link
Copy Markdown
Contributor

@monitz87 monitz87 left a comment

Choose a reason for hiding this comment

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

I left a few comments, but otherwise it looks good

Comment thread packages/new-polymath-issuer/src/pages/DividendsWizard/Step-3/index.tsx Outdated
Comment thread packages/new-polymath-issuer/src/pages/DividendsWizard/Step-3/index.tsx Outdated
@Grsmto Grsmto requested a review from monitz87 April 1, 2019 18:14
Copy link
Copy Markdown
Contributor

@monitz87 monitz87 left a comment

Choose a reason for hiding this comment

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

Looks good

@RafaelVidaurre RafaelVidaurre merged commit 896f40f into develop Apr 2, 2019
@RafaelVidaurre RafaelVidaurre deleted the feature/currency-select-disabled-tooltip branch April 2, 2019 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants