Skip to content

fix issue #894 #1194

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

Conversation

suppermancool
Copy link
Collaborator

fix issue #894
ABLE TO ADD INVALID OR INVALID FORMAT WEB URL IN ADD A WEB LINK.

@ThomasKranitsas
Copy link
Contributor

@suppermancool this is not a correct way to do it. Now google.com is treated as an invalid URL while is valid.

Also, the isWebLinkExist function is causing the app to crash now. Can you please fix that?

@suppermancool
Copy link
Collaborator Author

@ThomasKranitsas isWebLinkExist is bind in constructor, so you need to refresh the page before testing. Anyway i will change isWebLinkExist to webLinkExist in this PR, and fix the logic for url, pls wait until i finish

fix regex for url link, change isWebLinkExist -> webLinkExist
@suppermancool
Copy link
Collaborator Author

@ThomasKranitsas i fixed the regex for url and change isWebLinkExist to webLinkExist, pls check, thanks

@ThomasKranitsas
Copy link
Contributor

screen shot 2018-08-15 at 19 01 57

@suppermancool please see the above screenshot

@ThomasKranitsas
Copy link
Contributor

Account: TonyJ/appirio123

fix with some account link.URL is null
@suppermancool
Copy link
Collaborator Author

@ThomasKranitsas fixed, pls check again

@ThomasKranitsas ThomasKranitsas merged commit 2920580 into topcoder-platform:new-develop Aug 15, 2018
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