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

Redirect to original page after login #101

Open
zcesur opened this issue Sep 19, 2024 · 10 comments · May be fixed by #110
Open

Redirect to original page after login #101

zcesur opened this issue Sep 19, 2024 · 10 comments · May be fixed by #110

Comments

@zcesur
Copy link
Member

zcesur commented Sep 19, 2024

Currently, when a user logs in they are redirected to get_session(conn, :user_return_to), but in some flows the user_return_to value is never set

Example:

  • navigate to /ads (logged out)
  • get redirected to sign in
  • sign in
  • get redirected to / instead of /ads

Let's make sure every flow that takes the user to the login page also sets the user_return_to session value

@zcesur
Copy link
Member Author

zcesur commented Sep 19, 2024

/bounty $25

Copy link

algora-pbc bot commented Sep 19, 2024

💎 $25 bounty • Algora

Steps to solve:

  1. Start working: Comment /attempt #101 with your implementation plan
  2. Submit work: Create a pull request including /claim #101 in the PR body to claim the bounty
  3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

❗ Important guidelines:

  • To claim a bounty, you need to provide a short demo video of your changes in your pull request
  • If anything is unclear, ask for clarification before starting as this will help avoid potential rework
  • For assistance or questions, join our Discord

Thank you for contributing to algora-io/tv!

Add a bountyShare on socials

Attempt Started (GMT+0) Solution
🟢 @vikashsprem Sep 19, 2024, 3:34:53 PM WIP
🟢 @divanshu-go Sep 27, 2024, 5:00:25 PM #110
🔴 @saisandeepramavath Oct 8, 2024, 6:27:27 PM WIP
🟢 @melmathari Nov 24, 2024, 7:21:51 PM #120

@vikashsprem
Copy link

vikashsprem commented Sep 19, 2024

/attempt #101

Algora profile Completed bounties Tech Active attempts Options
@vikashsprem 1 bounty from 1 project
TypeScript, JavaScript,
Go & more
Cancel attempt

@divanshu-go
Copy link

divanshu-go commented Sep 27, 2024

/attempt #101

Algora profile Completed bounties Tech Active attempts Options
@bryans-go 4 bounties from 1 project
Go
Cancel attempt

divanshu-go added a commit to divanshu-go/tv that referenced this issue Sep 27, 2024
@divanshu-go divanshu-go linked a pull request Sep 27, 2024 that will close this issue
Copy link

algora-pbc bot commented Sep 27, 2024

💡 @bryans-go submitted a pull request that claims the bounty. You can visit your bounty board to reward.

divanshu-go added a commit to divanshu-go/tv that referenced this issue Sep 27, 2024
divanshu-go added a commit to divanshu-go/tv that referenced this issue Sep 27, 2024
@saisandeepramavath
Copy link

/attempt #101

@GauravBurande
Copy link

GauravBurande commented Oct 10, 2024

Currently, when a user logs in they are redirected to get_session(conn, :user_return_to), but in some flows the user_return_to value is never set

Example:

  • navigate to /ads (logged out)
  • get redirected to sign in
  • sign in
  • get redirected to / instead of /ads

Let's make sure every flow that takes the user to the login page also sets the user_return_to session value

so that's a redirect callback URL error, I see a few commits already, but if the issue is still unresolved, I can work on it.

@sky-39
Copy link

sky-39 commented Oct 31, 2024

Is this issue still relevant?

@melmathari
Copy link

melmathari commented Nov 24, 2024

/attempt #101

Copy link

algora-pbc bot commented Nov 24, 2024

💡 @melmathari submitted a pull request that claims the bounty. You can visit your bounty board to reward.

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.

7 participants