Skip to content

Bring autoconfigure notification a step forwards in custom events cre…#5912

Merged
aerosol merged 4 commits into
masterfrom
sanne-custom-events-autoconfigure
Dec 3, 2025
Merged

Bring autoconfigure notification a step forwards in custom events cre…#5912
aerosol merged 4 commits into
masterfrom
sanne-custom-events-autoconfigure

Conversation

@sanne-san

@sanne-san sanne-san commented Nov 26, 2025

Copy link
Copy Markdown
Contributor

…ation flow

Changes

  • Rather than showing a notification that custom events have been detected at the bottom of the form, we now show a modal prior to the form, that allows the user to add them instantly or set them up manually.

Before:

CleanShot 2025-11-26 at 11 28 22@2x

After:

CleanShot.2025-11-26.at.14.14.33.mp4

Tests

  • Automated tests have been added

Documentation

  • This change does not need a documentation update

Dark mode

  • The UI has been tested both in dark and light mode

…ation flow

- Rather than showing a notification that custom events have been detected at the bottom of the form, we now show a modal prior to the form, that allows the user to add them instantly or set them up manually.
@github-actions

Copy link
Copy Markdown
Preview environment👷🏼‍♀️🏗️
PR-5912

@sanne-san sanne-san force-pushed the sanne-custom-events-autoconfigure branch from f437a99 to 83a162c Compare November 26, 2025 13:13
@sanne-san sanne-san requested a review from a team November 26, 2025 13:16
@aerosol

aerosol commented Dec 3, 2025

Copy link
Copy Markdown
Member

I have some minor/stylistic reservations I'd like to push here later today if that's OK.

@sanne-san

Copy link
Copy Markdown
Contributor Author

I have some minor/stylistic reservations I'd like to push here later today if that's OK.

Absolutely, thanks!

@aerosol

aerosol commented Dec 3, 2025

Copy link
Copy Markdown
Member

There's something fishy going on with those tests, investigating... Sorry it's taking so long.

@aerosol aerosol added this pull request to the merge queue Dec 3, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Dec 3, 2025
@aerosol aerosol added this pull request to the merge queue Dec 3, 2025
Merged via the queue into master with commit 98e0f72 Dec 3, 2025
16 checks passed
@aerosol aerosol deleted the sanne-custom-events-autoconfigure branch February 11, 2026 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants