Skip to content

Merge main#7800

Draft
ljowen wants to merge 148 commits intosaas-nextfrom
merge-main
Draft

Merge main#7800
ljowen wants to merge 148 commits intosaas-nextfrom
merge-main

Conversation

@ljowen
Copy link
Contributor

@ljowen ljowen commented Mar 10, 2026

What this PR does

Fixes #

This is a description of what I've done in this PR, especially explaining choices made to make the reviewer's job easier. If I don't replace this paragraph with an actual description, my PR will probably be ignored.

Test me

How should reviewers test this? (Hint: If you want to provide a test catalog item, create a Gist of its catalog JSON, add its url and your branch name to this url: http://ci.terria.io/<branch name>/#clean&<raw url of your gist> , and then paste that in the body of this PR)

Checklist

  • There are unit tests to verify my changes are correct or unit tests aren't applicable (if so, write quick reason why unit tests don't exist)
  • I've updated relevant documentation in doc/.
  • I've updated CHANGES.md with what I changed.
  • I've provided instructions in the PR description on how to test this PR.

ljowen and others added 24 commits March 5, 2026 17:08
- adjust share panel padding
- extract out advanced utils
- show different copy for share link vs story link
…bottom spacing

- Story builder remove leading space on "Remove all stories"
- Update info color to match designs, use theme colors for shareUrlWarning
@CLAassistant
Copy link

CLAassistant commented Mar 10, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
4 out of 5 committers have signed the CLA.

✅ pjonsson
✅ zoran995
✅ peterhassall
✅ ljowen
❌ dependabot[bot]
You have signed the CLA already but the status is still pending? Let us recheck it.

@ljowen ljowen marked this pull request as draft March 12, 2026 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants