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

Improve contribution workflow #370

Draft
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

pratikb64
Copy link
Contributor

@pratikb64 pratikb64 commented Mar 4, 2025

Copy link

cypress bot commented Mar 4, 2025

Wiki    Run #483

Run Properties:  status check failed Failed #483  •  git commit b03ffd9fe1 ℹ️: Merge f26f623e01a2b7b4244be4eab04ca46bc44bafe0 into 0fe81a1c12239c4cafd0fbe64201...
Project Wiki
Branch Review user-contri-improvements
Run status status check failed Failed #483
Run duration 01m 24s
Commit git commit b03ffd9fe1 ℹ️: Merge f26f623e01a2b7b4244be4eab04ca46bc44bafe0 into 0fe81a1c12239c4cafd0fbe64201...
Committer Pratik Badhe
View all properties for this run ↗︎

Test results
Tests that failed  Failures 5
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 0
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

Tests for review

@rmehta
Copy link
Member

rmehta commented Mar 5, 2025

Can you add screenshots in your PR?

@pratikb64
Copy link
Contributor Author

This is how the notification banner and badges in app/space switcher will look like
image
image

@rmehta
Copy link
Member

rmehta commented Mar 6, 2025

Some dieas.

Since I have to land on a page to discover if there are any contributions, just show on every page of an Wiki Space. For example "There are 50 pending reviews on ERPNext"

When I click on the banner, it can show me a modal of the pages where there are contributions. If I am on a page with pending reviews, then show "There are 3 pending reviews on this page". If none, show global pending or just no notification.

The contributions can show up on a separate tab on a page (only visible to administrators). On the tab, when you click on an item, you can show the changes in a modal with an "Approve" or "Reject" button.

Also check espresso for banner UI

cc @netchampfaris

@pratikb64
Copy link
Contributor Author

This is how that table is looking (ignore the black box). Pagination, approve and reject are working
Things remaining:

  • Show a specific space's patches
  • Show diff in modal
  • Confirmation modal before approve/reject
  • Dark theme styling
    image

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.

2 participants