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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validate Funder Contact Text Input #59

Closed
starpause opened this issue Oct 27, 2023 · 6 comments
Closed

Validate Funder Contact Text Input #59

starpause opened this issue Oct 27, 2023 · 6 comments
Assignees

Comments

@starpause
Copy link
Contributor

Journey / Context

Fiona 馃悿馃挵 wants to list a new bounty.

Issue

Funder contact information is not validated.

Image

Implementation Idea

Use regex to ensure the input is a valid email or url.

@AngelBlock
Copy link
Contributor

AngelBlock commented Oct 27, 2023

@starpause please specify rules for other fields validation

@starpause
Copy link
Contributor Author

@AngelBlock

regex for email https://regexr.com/3e48o

regex for url (every other field) https://regexr.com/39nr7

@starpause starpause assigned AngelBlock and unassigned starpause Oct 30, 2023
@Kisgus Kisgus transferred this issue from another repository Nov 16, 2023
@cudam321
Copy link
Collaborator

looks good to me
image

i guess it's overkill for now to validate input for twitter, discord & telegram

@starpause
Copy link
Contributor Author

We are taking contact out of the bounty creation flow and keeping it on the user profile update flow, so we don't need to validate here. It will be good to have validation when users are filling out their contact channels on their profiles so the work in this request will still be valuable.

@AngelBlock AngelBlock removed their assignment Dec 5, 2023
@RasmusTheOnlyOne
Copy link
Collaborator

Work Good

@starpause
Copy link
Contributor Author

@rostyslav-dzhuman please push

@Kisgus Kisgus closed this as completed Jan 7, 2024
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

No branches or pull requests

6 participants