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

SQL Zoo: SQL Zoo link is broken or not working #29163

Closed
2 of 3 tasks
Hossam-Allam opened this issue Dec 8, 2024 · 3 comments · Fixed by #29167
Closed
2 of 3 tasks

SQL Zoo: SQL Zoo link is broken or not working #29163

Hossam-Allam opened this issue Dec 8, 2024 · 3 comments · Fixed by #29167
Assignees

Comments

@Hossam-Allam
Copy link
Contributor

Checks

Describe your suggestion

The sql zoo link in the assignment leads to a blank page

Path

Ruby / Rails, Node / JS

Lesson Url

https://www.theodinproject.com/lessons/databases-sql-zoo

(Optional) Discord Name

No response

(Optional) Additional Comments

No response

@Captainmango
Copy link
Contributor

I have a solution for this and will raise a PR shortly.

The cause is that the sqlzoo.net URL forces a redirect which Edge and Firefox (the browsers I tested) do not seem to follow. I was able to get the page to render in Chrome, but only after clearing my local cache.

sqlzoo has a permanent link on their page that forces the latest revision of the site. However, this would become stale very quickly. Instead, it looks like they have an option of sending a URL param to turn the redirect behaviour off.

https://sqlzoo.net/w/index.php?title=SQL_Tutorial&redirect=no I tried this in all the browsers I tested and it works as expected.

@Captainmango
Copy link
Contributor

PR that addresses this issue here #29167

@wise-king-sullyman
Copy link
Member

@Hossam-Allam thank you for raising this issue!

@Captainmango thanks for putting up the PR, I'll assign you to this issue, but so that you know for the future we prefer people to not put up PRs for opened issues without being assigned.

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 a pull request may close this issue.

3 participants