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

Data export in Your Rights & Consent not working #10906

Closed
sqlik opened this issue Sep 18, 2022 · 3 comments
Closed

Data export in Your Rights & Consent not working #10906

sqlik opened this issue Sep 18, 2022 · 3 comments
Assignees
@julianlam
Labels
bug regression
Milestone
2.5.3

Comments

@sqlik
Copy link

sqlik commented Sep 18, 2022

NodeBB version

2.5.2

NodeBB git hash

No response

NodeJS version

No response

Installed NodeBB plugins

No response

Database type

MongoDB, Redis

Database version

No response

Exact steps to cause this issue

When the user tries to download his data, an error is shown
/user/{user}/consent

screenshot-1

It is also not possible to download from the administrator account.
screenshot-2

What you expected

User can download his data .json, .csv and .zip

What happened instead

No response

Anything else?

No response
@sqlik sqlik added the bug label Sep 18, 2022
@barisusakli barisusakli added this to the 2.5.3 milestone Sep 18, 2022
@barisusakli
Copy link
Member

Regression from d057051

@julianlam julianlam modified the milestones: 2.5.3, 2.5.4 Sep 20, 2022
@julianlam julianlam reopened this Sep 20, 2022
@julianlam
Copy link
Member

julianlam commented Sep 20, 2022 •
edited
Loading

The url provided in the notification was not properly updated.

@julianlam julianlam modified the milestones: 2.5.4, 2.5.3 Sep 20, 2022
@julianlam
Copy link
Member

Ah, no, the old route still exists, but is just deprecated.

julianlam added a commit that referenced this issue Sep 20, 2022
@julianlam
fix: #10906, allow middleware.checkAccountPermissions to be called …
ee21731 
…with either uid or userslug in params

Previously, the middleware only worked with userslug params
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@julianlam julianlam

Labels
bug regression
Projects
None yet
Milestone
2.5.3
Development

No branches or pull requests
3 participants
@julianlam @barisusakli @sqlik