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

Fix a couple ProductAreaDropdown bugs #210

Merged
merged 8 commits into from
Jun 9, 2023
Merged

Conversation

jeffdaley
Copy link
Contributor

@jeffdaley jeffdaley commented Jun 9, 2023

  • Adds validation to the form when you select a product (i.e., the "Create Doc on Google" enables on ProductAreaSelect)
  • Saves the productAbbreviation when creating a new doc (by way of new attributes support in X::DropdownList
  • Improves the revert-on-error logic

@jeffdaley jeffdaley marked this pull request as ready for review June 9, 2023 15:34
@jeffdaley jeffdaley requested a review from a team as a code owner June 9, 2023 15:34
});
} catch (err) {
// revert field value on failure
targetField = oldVal;
(this as any)[field] = val;
console.error(err);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you mean to leave this in or was it just for debugging locally?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Intentional, but it makes more sense as a flashMessage. PR'd here: #211

@jeffdaley jeffdaley merged commit ef2a0ac into main Jun 9, 2023
@jeffdaley jeffdaley deleted the jeffdaley/product-area-fix branch June 9, 2023 21:35
anuragprafulla pushed a commit to razorpay/hermes that referenced this pull request Jun 27, 2023
* Support abbreviations

* Save productAbbreviation to doc; cleanup Save task

* Log Save error

* Fix abbreviation saving and broken tests

* Update comment

* Removes unnecessary abbreviation patch

* Add note about productAbbreviation
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