- Issue created by @tedbow
- Assigned to abhisekmazumdar
- Merge request !288#3471026: Validate UUID format and ensure uniqueness in the tree. → (Open) created by abhisekmazumdar
- Issue was unassigned.
- Status changed to Needs review
7 months ago 12:44pm 10 September 2024 - Assigned to tedbow
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Thanks @abhisekmazumdar!
@tedbow Thoughts on all the test-only UUIDs that now cause failures, such as
+ 'tree[uuid-in-root][the_body][0][uuid]' => 'The UUID "<em class="placeholder">uuid-level-1</em>" is not a valid UUID.' + 'tree[uuid-level-2][the_body][0][uuid]' => 'The UUID "<em class="placeholder">uuid-level-3</em>" is not a valid UUID.' + 'tree[uuid-level-2][the_body][1][uuid]' => 'The UUID "<em class="placeholder">uuid-last-in-tree</em>" is not a valid UUID.' + 'tree[uuid-level-1][the_body][0][uuid]' => 'The UUID "<em class="placeholder">uuid-level-2</em>" is not a valid UUID.'
?
- Issue was unassigned.
- Status changed to Postponed
7 months ago 8:16am 12 September 2024 - 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
LGTM! Well done, @abhisekmazumdar 😊 That looks as simple/elegant as I'd hoped 😄
📌 Update default config to make a fresh install result in an XB UI with an empty canvas Fixed will change/reduce the amount of test expectations that need to be updated, so postponing this on that.
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Didn't happen at Barcelona, but let's get it on https://contribkanban.com/board/experience_builder now 😄
- Assigned to wim leers
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Given the huge range of tests that would need updating by now, I don't think this is anymore.
It's something that might be a stable blocker. Doing it now doesn't buy us much though, so postponing.