Overview
- #3489743: Create AutoSave service and HTTP API to retrieve all entities with pending changes added a controller to provide the list of current changes
- The goal of this issue is to implement the "Review N changes" button on the front-end, which when clicked, shows which entities are autosaved but not yet published.
- The "Publish All" button is not part of the scope of this issue. #3497530: Implement the "Publish All" button is the issue for that, which is postponed on this one.
Proposed resolution
Create an api slice to `getAllPendingChanges` that reads from the controller added in #3489743: Create AutoSave service and HTTP API to retrieve all entities with pending changes.
User interface changes



| Comment | File | Size | Author |
|---|---|---|---|
| #32 | Screenshot 2025-01-23 at 12.16.38 PM.png | 39.06 KB | lauriii |
| #22 | Screenshot 2025-01-14 at 1.06.19 PM.png | 23.44 KB | lauriii |
| #15 | review-N-Changes.gif | 620.83 KB | soaratul |
| #11 | Screenshot from 2025-01-13 15-12-01.png | 52.97 KB | larowlan |
Issue fork experience_builder-3491459
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
larowlanComment #3
lauriiiComment #4
soaratul commentedComment #6
soaratul commentedComment #7
hooroomooLeft some comments
Comment #8
effulgentsia commentedI updated this issue's title and description to reflect a smaller scope: just the "Review N changes" button and what it shows when you click it. I opened #3497530: Implement the "Publish All" button for the "Publish All" button.
Comment #9
effulgentsia commentedIt looks to me like there are still some unresolved comments on the MR, so setting to "Needs work".
Comment #10
soaratul commentedComment #11
larowlanAdded screenshot to issue summary
Comment #12
wim leersI'd like to get a short screencast or a GIF in place to demonstrate this, because a static screenshot makes it hard to get a sense of the impact. 🙏
The (static) screenshots do look interesting already though! 😄
(Also a few high-level questions. Also: random failures on PostgreSQL CI runs, re-queued those 👍)
Comment #13
soaratul commentedComment #14
soaratul commentedComment #15
soaratul commentedComment #16
soaratul commentedComment #18
larowlanThere's one remaining thread here that I think can be resolved by just removing that piece of code, it seems to work fine without it.
Comment #19
larowlan@effulgentsia asked me to make that change and merge this so that @soaratul can continue with #3497530: Implement the "Publish All" button
Comment #21
larowlanCommitted to 0.x - thanks!
Comment #22
lauriiiWe might need some follow-ups for this if these don't already exist:
Comment #23
wim leersRather than creating follow-ups, let's just do another MR in this issue: to get the scope of this issue solved here.
Comment #24
jessebaker commentedUsing the same numbers as #22
Comment #25
lauriiiCorrect 👍
Ah you're right that this is actually a node and it should use the CMS icon. I think we should also come up with a better fallback icon.
+1 for a follow-up.
Sounds like something we should probably figure out in a follow-up.
Comment #26
larowlanOpened #3500390: The pending changes API endpoint should list individual regions for global template changes for follow-up
Comment #27
effulgentsia commentedUp until now, the XB team has been following a pseudo-scrum/pseudo-kanban process, but we're now shifting into more conventional scrum. We started a new 2-week sprint last Thursday (Jan 16). I'm tagging our current sprint's issues for visibility.
Comment #28
longwaveOpened #3501449: [Needs design] Decide how to assign colors to users for "review changes" for #22.4 / #24.4.
I think we could finish off #22.1 and #22.2 in this issue.
Comment #30
longwaveMR!577 implements #24.1/#24.2 and the comment in #25.
The last one wasn't available in Radix so I exported the SVG from Figma and added
vite-plugin-svgrto be able to use SVGs as React components - looks like this is the first time we have needed to do this.Comment #32
lauriiiThe icons and colors are working as expected now:
Comment #34
jessebaker commentedI've approved and merged !577 which resolves #24.1 and #24.2
Closing this issue. There are two follow ups for #24.3 and #24.4
#3500390: The pending changes API endpoint should list individual regions for global template changes and
#3501449: [Needs design] Decide how to assign colors to users for "review changes"
Comment #35
effulgentsia commented