Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add additional message fields to promotion #190

Merged
merged 2 commits into from
Feb 28, 2023
Merged

Conversation

vannguyenn
Copy link
Collaborator

@vannguyenn vannguyenn commented Feb 9, 2023

Resolves #175 #176
Related changes on API:
reactioncommerce/reaction#6788

Testing Instructions:

  • Start the app and login with a valid account
  • Navigate to Promotions
  • Click on one promotion row to go to a Promotion details page
  • Scroll down to Promotion Message section

image

Signed-off-by: Chloe <pinkcloudvnn@gmail.com>
@tedraykov
Copy link
Collaborator

Not related to this PR's changes but when I try to create a promotion without setting the Max $ discount on an order, I don't get a form field validation error in the UI, but we still get a response error that the field is required.

This may be fixed in the rest of the PRs but I want to add a note here so we don't miss this out if it isn't handled.

Screenshot 2023-02-27 at 12 23 50

Screenshot 2023-02-27 at 12 24 07

@vannguyenn
Copy link
Collaborator Author

Thanks @tedraykov, issues with promotion limit usages will be fixed in another PR

@vannguyenn vannguyenn merged commit 458ea68 into trunk Feb 28, 2023
@vannguyenn vannguyenn deleted the feat/additional-fields branch February 28, 2023 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants