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

Failing to add a url to a sponsor #5179

Closed
datapythonista opened this issue Sep 23, 2020 · 7 comments · Fixed by #5185
Closed

Failing to add a url to a sponsor #5179

datapythonista opened this issue Sep 23, 2020 · 7 comments · Fixed by #5185

Comments

@datapythonista
Copy link

I'm adding a url to a sponsor in https://eventyay.com/events//edit/sponsors and the error "Not a valid url" is returned, and saving fails. I tried with different urls, including https://eventyay.com

Thanks!

@iamareebjamal
Copy link
Member

Test-Event-Events-eventyay-Googl

Hi, I tried adding the URL successfully. Do you type the URLs or copy-paste? I tried both with and without https://

Can you add some screenshots or a video?

@iamareebjamal
Copy link
Member

iamareebjamal commented Sep 29, 2020

As the https:// part is already a prefix, please ensure you are not typing https:// in the field itself. When you paste the URL in the field, it is taken care of automatically

@datapythonista
Copy link
Author

Screenshot at 2020-09-29 17-02-43

Now I'm having a different error. I do have a ticket created, and several orders.

@iamareebjamal
Copy link
Member

Thank you. Let me check

@iamareebjamal
Copy link
Member

The above PR has most probably fixed the error, so can you please try https://open-event-frontend-h0ws9ciu8.vercel.app/ and check if you are still getting error while saving sponsors

@datapythonista
Copy link
Author

When are the PRs you're merging being released? Our event is on Thursday, but I'll give it a try again once released.

@iamareebjamal
Copy link
Member

They're released about 15 minutes after merging, so this one is already live. The above URL I gave was also linked to production instance with the new changes so that could be used to test changes as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants