Skip to content

Do not show errors on push notifications fields when shown for the first time after validation.#1492

Merged
dompuiu merged 2 commits into
mainfrom
fix_push_notifications_fields_showing_errors_on_initial_show
May 1, 2026
Merged

Do not show errors on push notifications fields when shown for the first time after validation.#1492
dompuiu merged 2 commits into
mainfrom
fix_push_notifications_fields_showing_errors_on_initial_show

Conversation

@dompuiu
Copy link
Copy Markdown
Member

@dompuiu dompuiu commented May 1, 2026

Changed Packages

  • core
  • reactor-extension

Description

Push notification fields were showing validation errors when first enabled after the form was previously validated while the component was disabled.

Related Issue

Motivation and Context

Screenshots (if appropriate):

Documentation

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Improvement (non-breaking change which does not add functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have added a Changeset file with a consumer-facing description of my changes.
  • I have signed the Adobe Open Source CLA or I'm an Adobe employee.
  • I have made any necessary test changes and all tests pass.
  • I have run the Sandbox successfully.

@dompuiu dompuiu self-assigned this May 1, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 1, 2026

🦋 Changeset detected

Latest commit: e610412

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
reactor-extension-alloy Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@dompuiu dompuiu merged commit b50a444 into main May 1, 2026
11 checks passed
@dompuiu dompuiu deleted the fix_push_notifications_fields_showing_errors_on_initial_show branch May 1, 2026 16:39
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.

2 participants