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

Remove misleading currency caption #3620

Merged
merged 8 commits into from
Jun 6, 2023
Merged

Remove misleading currency caption #3620

merged 8 commits into from
Jun 6, 2023

Conversation

timuric
Copy link
Contributor

@timuric timuric commented May 10, 2023

Remove misleading description

PR intended to be tested with API branch:

Screenshots

Voucher screen

Before:
image

After :
image

Before:
image

After:
image

Before:
image

After:
image

Before:
image

After:
image

Before:
image

After:
image

Pull Request Checklist

  1. This code contains UI changes
  2. All visible strings are translated with proper context including data-formatting
  3. Attributes [data-test-id] are added for new elements
  4. Changes are mentioned in the changelog
  5. The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://automation-dashboard.staging.saleor.cloud/graphql/
APPS_MARKETPLACE_API_URI=https://apps.staging.saleor.io/api/v2/saleor-apps

Do you want to run more stable tests?

To run all tests, just select the stable checkbox. To speed up tests, increase the number of containers. Tests will be re-run only when the "run e2e" label is added.

  1. stable
  2. giftCard
  3. category
  4. collection
  5. attribute
  6. productType
  7. shipping
  8. customer
  9. permissions
  10. menuNavigation
  11. pages
  12. sales
  13. vouchers
  14. homePage
  15. login
  16. orders
  17. products
  18. app
  19. plugins
  20. translations
  21. navigation
  22. variants
  23. payments

CONTAINERS=1

@github-actions github-actions bot temporarily deployed to copy-adjustments May 10, 2023 10:16 Inactive
@github-actions github-actions bot requested a deployment to storybook copy-adjustments May 10, 2023 10:16 In progress
@github-actions github-actions bot temporarily deployed to copy-adjustments May 10, 2023 11:09 Inactive
@github-actions github-actions bot temporarily deployed to storybook copy-adjustments May 10, 2023 11:09 Inactive
@timuric timuric requested a review from andrzejewsky May 10, 2023 11:17
@timuric timuric self-assigned this May 10, 2023
@timuric timuric changed the title Remove misleading voucher value caption Remove misleading currency caption May 10, 2023
@github-actions github-actions bot temporarily deployed to copy-adjustments May 10, 2023 11:19 Inactive
@github-actions github-actions bot temporarily deployed to storybook copy-adjustments May 10, 2023 11:19 Inactive
@github-actions github-actions bot temporarily deployed to copy-adjustments May 10, 2023 11:20 Inactive
@github-actions github-actions bot temporarily deployed to storybook copy-adjustments May 10, 2023 11:21 Inactive
andrzejewsky
andrzejewsky previously approved these changes May 11, 2023
krzysztofzuraw
krzysztofzuraw previously approved these changes May 11, 2023
@michalina-graczyk michalina-graczyk self-assigned this May 15, 2023
@michalina-graczyk
Copy link
Member

michalina-graczyk commented May 15, 2023

image
Shouldn't it be removed also from this view? @timuric

@timuric timuric dismissed stale reviews from krzysztofzuraw and andrzejewsky via c60f2d3 May 30, 2023 09:40
@timuric timuric requested a review from a team May 30, 2023 09:40
krzysztofzuraw
krzysztofzuraw previously approved these changes May 30, 2023
@timuric
Copy link
Contributor Author

timuric commented May 31, 2023

@michalina-graczyk good catch! Removed it as well

@andrzejewsky andrzejewsky merged commit f5f216e into main Jun 6, 2023
@andrzejewsky andrzejewsky deleted the copy-adjustments branch June 6, 2023 12:27
andrzejewsky added a commit that referenced this pull request Jun 20, 2023
* Remove misleading voucher value caption

* Remove misleading order value caption

* Remove misleading sale value caption

* Remove misleading pricing card value caption

* Remove misleading variant pricing card value caption

* Extract messages

* Update messages

---------

Co-authored-by: andrzejewsky <vox3r69@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

5 participants