Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix links validations #979

Merged
merged 1 commit into from
Jun 9, 2020
Merged

Fix links validations #979

merged 1 commit into from
Jun 9, 2020

Conversation

BenGamma
Copy link
Contributor

@BenGamma BenGamma commented Jun 9, 2020

PR Type

  • [ x ] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

PR Checklist

  • - Latest master branch merged
  • - PR title descriptive (can be used in release notes)
  • - Passes Tests

Description

I've spotted a weird behavior in the link validation while doing a last check. See a screencast here
It happend in the case you have an invalid url first, then the validation removes the last char of the input. But I don't get it because we are not striping any character, we are just adding from the existing value 🤔

@cypress
Copy link

cypress bot commented Jun 9, 2020



Test summary

60 0 0 0


Run details

Project onearmy-community-platform
Status Passed
Commit a5b7e72 ℹ️
Started Jun 9, 2020 10:56 AM
Ended Jun 9, 2020 11:00 AM
Duration 04:27 💡
OS Linux Ubuntu Linux - 16.04
Browser Multiple

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@BenGamma
Copy link
Contributor Author

BenGamma commented Jun 9, 2020

The bug seems to come from the url validation validate={value => this.validateDependingOnType(value)}. But it's very weird, sometimes it append, sometimes not. Can you let me know if you can reproduce @chrismclarke ?

@chrismclarke chrismclarke marked this pull request as ready for review June 9, 2020 16:49
@chrismclarke
Copy link
Member

I tried copying the steps exactly but can't recreate on my local browsers, so I think perhaps we can just put this down to a browser quirk and assume if it comes up again the user will be able to resolve themselves. Will merge the discord change

@chrismclarke chrismclarke merged commit 0e48929 into master Jun 9, 2020
@chrismclarke chrismclarke deleted the fix/818-profile-double-type branch June 9, 2020 16:50
@BenGamma BenGamma mentioned this pull request Jul 13, 2020
@BenGamma BenGamma mentioned this pull request Sep 30, 2020
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.

2 participants