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 handling of account update API response #7476

Merged
merged 1 commit into from
Oct 24, 2022
Merged

Fix handling of account update API response #7476

merged 1 commit into from
Oct 24, 2022

Conversation

ChristophWurst
Copy link
Member

Axios gives us the response contents inside data, but our response also has a key data. So we need .data.data.

Found in #7430

@ChristophWurst ChristophWurst self-assigned this Oct 21, 2022
@ChristophWurst
Copy link
Member Author

/backport to stable2.1

@ChristophWurst ChristophWurst mentioned this pull request Oct 21, 2022
Merged
9 tasks
miaulalala
miaulalala approved these changes Oct 24, 2022
View reviewed changes
Copy link
Contributor

@miaulalala miaulalala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

@ChristophWurst ChristophWurst merged commit 385680d into main Oct 24, 2022
@ChristophWurst ChristophWurst deleted the fix/account-update-response-handling branch October 24, 2022 16:22
@backportbot-nextcloud backportbot-nextcloud bot mentioned this pull request Oct 24, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@miaulalala miaulalala miaulalala approved these changes

@st3iny st3iny Awaiting requested review from st3iny

@GretaD GretaD Awaiting requested review from GretaD GretaD is a code owner

@kesselb kesselb Awaiting requested review from kesselb kesselb is a code owner

Assignees

@ChristophWurst ChristophWurst

Labels
3. to review bug
Projects
💌 📅 👥 Groupware team
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ChristophWurst @miaulalala