Skip to content

Settings Manager amendments#182

Open
akaia-shadowfox wants to merge 601 commits into
mainfrom
settings-editor-amendments
Open

Settings Manager amendments#182
akaia-shadowfox wants to merge 601 commits into
mainfrom
settings-editor-amendments

Conversation

@akaia-shadowfox

Copy link
Copy Markdown
Contributor

No description provided.

@lennczar

lennczar commented Jan 23, 2023

Copy link
Copy Markdown
Contributor

A few changes were requested in an internal meeting. The requests have been transcribed into github issues.

As #190 & #191 tackle issues that are already on main, they can be resolved after the merge

@akaia-shadowfox

Copy link
Copy Markdown
Contributor Author

A bug that should be fixed before the merge:

  1. Open “app” page
  2. Select a DAO where you have the proposal creation permission and reload the page. If it's selected already, just reload the page
  3. Create any valid workflow, fill the proposal description as well
  4. Check the “Propose on …” button. It'll probably be blocked with 403 error
  5. While staying on the page, switch to another DAO where you have the proposal creation permission
  6. Check the “Propose on …” button again
  7. Switch back to the previous DAO
  8. Check the “Propose on …” button again
  9. Now, proposal creation is not blocked

@akaia-shadowfox

Copy link
Copy Markdown
Contributor Author

A bug that should be fixed before the merge:

  1. Open “app” page
  2. Select a DAO where you have the proposal creation permission and reload the page. If it's selected already, just reload the page
  3. Create any valid workflow, fill the proposal description as well
  4. Check the “Propose on …” button. It'll probably be blocked with 403 error
  5. While staying on the page, switch to another DAO where you have the proposal creation permission
  6. Check the “Propose on …” button again
  7. Switch back to the previous DAO
  8. Check the “Propose on …” button again
  9. Now, proposal creation is not blocked

Seems fixed!

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