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

Remove remoteRedirect() for profile website fix #2996 #3031

Merged
merged 6 commits into from Dec 4, 2021

Conversation

samuelroland
Copy link
Contributor

As discussed in the related issue #2996, I've followed your indications @dansup, I've made the changes online, so I didn't test in a local env...

@dansup dansup merged commit 89d6217 into pixelfed:staging Dec 4, 2021
@dansup
Copy link
Member

dansup commented Dec 4, 2021

@samuelroland Thank you!

@samuelroland samuelroland deleted the remote-redirect-remove branch December 4, 2021 01:26
@samuelroland
Copy link
Contributor Author

Thanks for your help @dansup.
I wanted to contribute to other opensource projects than my own since a loooonng time, and I finally did it 🥳!

This was the first PR but certainly not the last. Feels great when it's merged! 😄
I will try to search other beginner issues to tackle in the next days (but now I'm going to bed, because it's 2AM for me, hum...).

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 this pull request may close these issues.

None yet

2 participants