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.

Sign up for GitHub

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

Jump to bottom

fix: redirect settings/profile #578

Closed
1 task done
aroualid opened this issue Feb 12, 2024 · 1 comment · Fixed by #579
Closed
1 task done

fix: redirect settings/profile #578

aroualid opened this issue Feb 12, 2024 · 1 comment · Fixed by #579
Assignees
@42atomys
Labels
aspect/interface 🕹 Concerns end-users' experience with the software domain/obvious 🟩 Represents the "known knowns" issue. It's Obviously priority/high 🟧 Priority 2 - Stalls work on the project or its dependents type/bug 🔥 Something isn't working

Comments

@aroualid
Copy link

Describe the bug

hi, when I connect discord, github or spotify, the website redirects me to "https://s42.app/settings/accounts" but the page dosen't exist, the correct redirection is "https://s42.app/settings/profile"

To Reproduce

try to connect spotify github or discord on "https://s42.app/settings/profile"

Expected behavior

No response

Relevant log output

No response

Version of software

v0.31.1

Environment

Sandbox (https://sandbox.s42.dev)

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@aroualid aroualid added state/triage 🚦 Has not been triaged & therefore, not ready for work type/bug 🔥 Something isn't working labels Feb 12, 2024
@42atomys 42atomys added priority/high 🟧 Priority 2 - Stalls work on the project or its dependents aspect/interface 🕹 Concerns end-users' experience with the software domain/obvious 🟩 Represents the "known knowns" issue. It's Obviously and removed state/triage 🚦 Has not been triaged & therefore, not ready for work labels Feb 12, 2024
@42atomys
Copy link
Owner

Hi @aroualid thanks for your report, I will works to fix that quickly 💜

@42atomys 42atomys self-assigned this Feb 18, 2024
@42atomys 42atomys mentioned this issue Feb 18, 2024
Merged
3 tasks
42atomys added a commit that referenced this issue Feb 18, 2024
@42atomys
fix: oauth2 redirects point to a 404 (#579)
8280d41 
**Describe the pull request**

This pull request reflect the new settings page URL on the oauth2
callback. Actually the callback redirect to a 404 Not Found error page.

Resolve #578 

**Checklist**

- [x] I have made the modifications or added tests related to my PR
- [x] I have run the tests and linters locally and they pass
- [x] I have added/updated the documentation for my RP
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@42atomys 42atomys

Labels
aspect/interface 🕹 Concerns end-users' experience with the software domain/obvious 🟩 Represents the "known knowns" issue. It's Obviously priority/high 🟧 Priority 2 - Stalls work on the project or its dependents type/bug 🔥 Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: oauth2 redirects point to a 404 42atomys/stud42
2 participants
@42atomys @aroualid