Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Bug] Review prompt prevents admins from deleting other user accounts #192

Closed
joe9663 opened this issue Jun 24, 2021 · 0 comments · Fixed by #218
Closed

[Bug] Review prompt prevents admins from deleting other user accounts #192

joe9663 opened this issue Jun 24, 2021 · 0 comments · Fixed by #218

Comments

@joe9663
Copy link

joe9663 commented Jun 24, 2021

Describe the bug
The 5 star review prompt looks to have some kind of nonce conflict with the Delete User functionality. Looks to be a similar situation to BoldGrid/boldgrid-backup#332. While the prompt is present you are directed to a WordPress error page stating The link you are following has expired when attempting to remove a user. Dismissing the notice allows you to delete users as expected.

jamesros161 added a commit that referenced this issue Sep 8, 2021
@jamesros161 jamesros161 linked a pull request Sep 8, 2021 that will close this issue
jamesros161 added a commit that referenced this issue Oct 25, 2021
* initial commit

* resolves #139 (#217)

* resolves #192 (#218)

* resolves #198 (#219)

* update versions and readme

* update versions for stable release

* update composer.json

* add intval to $_GET['post_id']

* fix: upgrade postcss-loader from 2.0.8 to 2.1.6 (#180)

Snyk has created this PR to upgrade postcss-loader from 2.0.8 to 2.1.6.

See this package in npm:


See this project in Snyk:
https://app.snyk.io/org/cssjoe/project/e00ad434-86a9-4713-b063-d21380ccacc6?utm_source=github&utm_medium=upgrade-pr

* fix: upgrade babel-eslint from 8.0.2 to 8.2.6 (#179)

Snyk has created this PR to upgrade babel-eslint from 8.0.2 to 8.2.6.

See this package in npm:


See this project in Snyk:
https://app.snyk.io/org/cssjoe/project/e00ad434-86a9-4713-b063-d21380ccacc6?utm_source=github&utm_medium=upgrade-pr

* fix: upgrade eslint from 4.11.0 to 4.19.1 (#178)

Snyk has created this PR to upgrade eslint from 4.11.0 to 4.19.1.

See this package in npm:


See this project in Snyk:
https://app.snyk.io/org/cssjoe/project/e00ad434-86a9-4713-b063-d21380ccacc6?utm_source=github&utm_medium=upgrade-pr

Co-authored-by: jamesros <jamesros@boldgrid.com>

* fix: upgrade webpack-dev-server from 3.1.1 to 3.11.2 (#177)

Snyk has created this PR to upgrade webpack-dev-server from 3.1.1 to 3.11.2.

See this package in npm:


See this project in Snyk:
https://app.snyk.io/org/cssjoe/project/e00ad434-86a9-4713-b063-d21380ccacc6?utm_source=github&utm_medium=upgrade-pr

Co-authored-by: jamesros <jamesros@boldgrid.com>

* update yarn.lock

Co-authored-by: Rafael Ramos <rafaelh@inmotionhosting.com>
Co-authored-by: Snyk bot <github+bot@snyk.io>
jamesros161 added a commit that referenced this issue Nov 18, 2021
* add full screen functionality

* update css and version numbers

* Add admin pointer for new feature (#240)

* Issue 229 (#238)

* adjust button styling

* resolves 229

* resolves #227 and #228 (#237)

* resolves 225, and fixes tooltip (#236)

* add publish buttons to fullscreen toolbar (#235)

* update version numbers and fix merge

* fix pointer not showing on page.

* Issue 243 (#247)

* redo styling of UI

* break to multiple lines on small screens

* resolves #250, #253, #245 (#255)

* resolves #248 (#256)

* Resolves #224, #249, and #251 (#259)

* remove error_log and console.log statements

* fix initial width

* fix pointer

* fix point repositioniing

* fix #224

* add 'secure' to travis.yml

* Merge October patch release changes into 1.15.0 (#278)

* initial commit

* resolves #139 (#217)

* resolves #192 (#218)

* resolves #198 (#219)

* update versions and readme

* update versions for stable release

* update composer.json

* add intval to $_GET['post_id']

* fix: upgrade postcss-loader from 2.0.8 to 2.1.6 (#180)

Snyk has created this PR to upgrade postcss-loader from 2.0.8 to 2.1.6.

See this package in npm:


See this project in Snyk:
https://app.snyk.io/org/cssjoe/project/e00ad434-86a9-4713-b063-d21380ccacc6?utm_source=github&utm_medium=upgrade-pr

* fix: upgrade babel-eslint from 8.0.2 to 8.2.6 (#179)

Snyk has created this PR to upgrade babel-eslint from 8.0.2 to 8.2.6.

See this package in npm:


See this project in Snyk:
https://app.snyk.io/org/cssjoe/project/e00ad434-86a9-4713-b063-d21380ccacc6?utm_source=github&utm_medium=upgrade-pr

* fix: upgrade eslint from 4.11.0 to 4.19.1 (#178)

Snyk has created this PR to upgrade eslint from 4.11.0 to 4.19.1.

See this package in npm:


See this project in Snyk:
https://app.snyk.io/org/cssjoe/project/e00ad434-86a9-4713-b063-d21380ccacc6?utm_source=github&utm_medium=upgrade-pr

Co-authored-by: jamesros <jamesros@boldgrid.com>

* fix: upgrade webpack-dev-server from 3.1.1 to 3.11.2 (#177)

Snyk has created this PR to upgrade webpack-dev-server from 3.1.1 to 3.11.2.

See this package in npm:


See this project in Snyk:
https://app.snyk.io/org/cssjoe/project/e00ad434-86a9-4713-b063-d21380ccacc6?utm_source=github&utm_medium=upgrade-pr

Co-authored-by: jamesros <jamesros@boldgrid.com>

* update yarn.lock

Co-authored-by: Rafael Ramos <rafaelh@inmotionhosting.com>
Co-authored-by: Snyk bot <github+bot@snyk.io>

* update built editor.min.js

* update version numbers

* update readme.txt

* add additional theme buttons

* additional theme buttons

* Add primary and secondary button fallbacks (#284)

* fix paragraph dropdown in full screen (#289)

* update fourpan package (#290)

* resolves issue 254 (#291)

* update version number for rc2

* remove phpunit temporarily

* update fourpan version

* fix contextToolbars

* remove console.log

* update version and readme

Co-authored-by: Rafael Ramos <rafaelh@inmotionhosting.com>
Co-authored-by: Snyk bot <github+bot@snyk.io>
Co-authored-by: Jesse Owens <jesse@jessecowens.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant