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

correct opt-out showing application #10086

Merged
merged 3 commits into from
Feb 20, 2019

Conversation

hinaloe
Copy link
Contributor

@hinaloe hinaloe commented Feb 20, 2019

in #9994, api response of status was not correct opted-out without save settings. (/@user/:status is correct.)

@Gargron
Copy link
Member

Gargron commented Feb 20, 2019

I am confused. User#shows_application? calls the wrong variable?

@hinaloe
Copy link
Contributor Author

hinaloe commented Feb 20, 2019

Oh, that right. User#shows_application? looks call wrong var.

I reverted change of serializer and fix User model.

@Gargron Gargron merged commit 4340d33 into mastodon:master Feb 20, 2019
hiyuki2578 pushed a commit to ProjectMyosotis/mastodon that referenced this pull request Oct 2, 2019
* correct opt-out showing application

refs mastodon#9994

* Revert "correct opt-out showing application"

This reverts commit 0e9bb70.

* User#shows_application? calls wrong value
messenjahofchrist pushed a commit to Origin-Creative/mastodon that referenced this pull request Jul 30, 2021
* correct opt-out showing application

refs mastodon#9994

* Revert "correct opt-out showing application"

This reverts commit 0e9bb70.

* User#shows_application? calls wrong value
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.

3 participants