openedx / openedx/frontend-app-authoring
[Cleanup/Followup] Remove update/ignore quick actions in library updates listing
Open
Beginner friendly
Nobody has claimed this yet.
mao-onboarding
- Dominant language
- TypeScript
- Stars
- 17
- Forks
- 218
- Avg merge
- 9d 20h
- Merged PRs (30d)
- 20
Description
| Current | Expected |
|---|---|
Spec
- Ignore and update buttons should be removed from the library updates listing view.
- Each available update should only show a button to review updates on the right side.
Rationale
This ticket is filed under https://github.com/openedx/frontend-app-authoring/issues/2182 because it is a prerequisite to clarifying local override behavior.
- Ignore/update should require reviewing changes before a user decides whether to ignore/update or take other actions. As we support overrides and higher-level library structural blocks, it's important to ensure users review changes before approving them.
- In the future, some content syncs will include more nuanced options than only "ignore" and "update" (e.g. choosing whether to keep local overrides). Dropping the shortcuts will allow the default/secondary option to vary per object for some cases.
In Scope
- Remove separator
- Remove ignore and update buttons
Out of scope
Any other changes
Related Issues
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the library updates listing view and compare it with the current and expected screenshots in the issue. Remove the separator and the Ignore and Update actions, leaving only the Review Updates button on the right; confirm that no other changes are included.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 68/100