Skip to content

Fix for issue# 2857 on connect-app - #313

Merged
RishiRajSahu merged 3 commits into
topcoder-platform:feature/connectv2from
akumar1503:fix-2857-connect-app
Mar 5, 2019
Merged

Fix for issue# 2857 on connect-app#313
RishiRajSahu merged 3 commits into
topcoder-platform:feature/connectv2from
akumar1503:fix-2857-connect-app

Conversation

@akumar1503

Copy link
Copy Markdown
  • Bottom link is now clickable in WizardRight.
  • Continue button fix in RegistrationScreen, it's not required to click the "I agree to receive other communications from Topcoder." checkbox.
  • removed + sign requirement in phone numbers in Formsy.PhoneInput.

…egistrationScreen and remove + sign requirement in phone numbers in Formsy.PhoneInput

@maxceem maxceem left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Everything works good @akumar1503 .

Only one thing, could you please return adding + when we choose the county code in the dropdown.
We don't require entering + by users, but we still want to add + when we choose the country code.

@akumar1503

Copy link
Copy Markdown
Author

@maxceem done.

@akumar1503

Copy link
Copy Markdown
Author

added minor correction, done now.

@maxceem maxceem left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Works good!

Thank you @akumar1503

@maxceem

maxceem commented Feb 28, 2019

Copy link
Copy Markdown
Collaborator

@RishiRajSahu I've tested this PR to cover 3 issues listed here https://github.com/appirio-tech/connect-app/issues/2857

It works good for me.
It can be merged separately from changes in Connect app.

@RishiRajSahu
RishiRajSahu merged commit 4389bfe into topcoder-platform:feature/connectv2 Mar 5, 2019
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