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

[Products] Empty option on unit scale dropdown #12969

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chahmedejaz
Copy link
Collaborator

@chahmedejaz chahmedejaz commented Nov 9, 2024

What? Why?

What should we test?

  • Add a new variant for the product on the products page
  • Please make sure that you can view "Select unit scale" as first option for the unit scale field.

Release notes

Changelog Category (reviewers may add a label for the release notes):

  • User facing changes
  • API changes (V0, V1, DFC or Webhook)
  • Technical changes only
  • Feature toggled

@chahmedejaz chahmedejaz changed the title 12919: remove empty option from unit scale dropdown [Products page] Empty option on unit scale dropdown Nov 9, 2024
@chahmedejaz chahmedejaz added bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. user facing changes Thes pull requests affect the user experience labels Nov 9, 2024
@chahmedejaz chahmedejaz force-pushed the task/12919-remove-empty-option-from-unit-scale-dropdown branch from cd6b2ae to bafb881 Compare November 10, 2024 09:58
@chahmedejaz chahmedejaz marked this pull request as draft November 10, 2024 10:02
@chahmedejaz chahmedejaz marked this pull request as ready for review November 11, 2024 23:13
Copy link
Member

@dacook dacook left a comment

Choose a reason for hiding this comment

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

From what I can see, the include_blank option was added here, perhaps to handle bad data.

Does this mean that every dropdown now has a blank "select unit scale" option? I think it would be ideal if we show "select unit scale" only when data is blank. Is that possible?

@sigmundpetersen sigmundpetersen changed the title [Products page] Empty option on unit scale dropdown [Products] Empty option on unit scale dropdown Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. user facing changes Thes pull requests affect the user experience
Projects
Status: Code review 🔎
Development

Successfully merging this pull request may close these issues.

[Products page] Empty option on unit scale dropdown
2 participants