Closed (fixed)
Project:
Experience Builder
Version:
0.x-dev
Component:
Page builder
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
3 Dec 2024 at 21:56 UTC
Updated:
14 Feb 2025 at 04:49 UTC
Jump to comment: Most recent, Most recent file



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