Ensure segments have a weight

How does this relate to 📌 Provide a locked Default segment Active ? Will that have a special reserved weight that no other Segment can set?

I'm guessing we will be giving that one a high value, or ensuring all locked segments come last?

I've created an MR for adding weight to Segments, I haven't addressed anything in the Drupal UI for weighting as I saw this issue: 📌 Revisit segments UI: do we need a Drupal UI? Active and I assume it may get addressed there?

I've also not addressed discussion in #2/#3 as seems thats not currently confirmed. But we might need 📌 Provide a locked Default segment Active to land first? Or it get handled in that ticket after this lands?

I think that's all we need. We can discuss how do we want to handle #2 in 📌 Provide a locked Default segment Active .

Looks good!

Two bugs:

1. weight is not exposed by the HTTP API, which AFAICT will make the UI impossible to implement: https://git.drupalcode.org/project/experience_builder/-/merge_requests/1...
2. weight should either be updated via the HTTP API, or it should have a comment pointing to which issue will make it so: https://git.drupalcode.org/project/experience_builder/-/merge_requests/1...

As Wim pointed out I missed we need to expose the weight to the client-side. Which means openapi.yml needs updates too.

This looks great now, thanks! 😊

Just one last bug in the OpenAPI spec: https://git.drupalcode.org/project/experience_builder/-/merge_requests/1...

One last bit was missing, fixed that. Awaiting green CI results 🤞

Actually, as I was about to merge, all the weight: 0 lines that were deleted in https://git.drupalcode.org/project/experience_builder/-/merge_requests/1... should be restored 😅 They were totally right — without those, the examples do would not be valid against their (now updated) schema!

Restored all deleted lines in https://git.drupalcode.org/project/experience_builder/-/merge_requests/1...

Again awaiting green CI results, then merging.

Automatically closed - issue fixed for 2 weeks with no activity.