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

Link Login Modal to Comments page on notes #4156

Closed
SidharthBansal opened this issue Dec 7, 2018 · 9 comments
Closed

Link Login Modal to Comments page on notes #4156

SidharthBansal opened this issue Dec 7, 2018 · 9 comments

Comments

@SidharthBansal
Copy link
Member

We are creating a login/signup modal, next step is to link to various pages. In this issue we will be linking it to Comments page on notes
publiclab.org/notes/
app/views/notes/_comments.html.erb
screenshot 54

@SidharthBansal
Copy link
Member Author

This task is published on GCI dashboard. Thanks all.

@oorjitchowdhary
Copy link
Member

@SidharthBansal I've solved this one.. but it is not redirecting to the origin page.. so should I write the code for that in my patch or just send the patch without the redirect... We've merged the return_to PR right?

@SidharthBansal
Copy link
Member Author

Yes return_to PR is merged.
You need to made it redirect to the same page somehow.

@oorjitchowdhary
Copy link
Member

oorjitchowdhary commented Dec 9, 2018

how do I make it redirect.. I'm not so familiar with the redirect code

@SidharthBansal
Copy link
Member Author

SidharthBansal commented Dec 9, 2018

Please first push your pr so that I can review the gif etc.

@oorjitchowdhary
Copy link
Member

Okay sure..

@oorjitchowdhary
Copy link
Member

@SidharthBansal the return_to is working now.. Actually, I hadn't made the changes which was to be made after #4197

@oorjitchowdhary
Copy link
Member

So I'm sending a patch

@SidharthBansal
Copy link
Member Author

SidharthBansal commented Dec 9, 2018 via email

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

No branches or pull requests

2 participants