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

Export products as csv file test #4521

Merged
merged 15 commits into from
Dec 14, 2023
Merged

Export products as csv file test #4521

merged 15 commits into from
Dec 14, 2023

Conversation

wojteknowacki
Copy link
Member

@wojteknowacki wojteknowacki commented Dec 7, 2023

Export products as csv file test

Screenshots

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. The changes are tested in Chrome/Firefox/Safari browsers and in light/dark mode
  5. Your code works with the latest stable version of the core
  6. I added changesets file (instructions in contribution guide

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

CONTAINERS=2

@wojteknowacki wojteknowacki requested review from a team, andrzejewsky and poulch December 7, 2023 09:14
Copy link

changeset-bot bot commented Dec 7, 2023

🦋 Changeset detected

Latest commit: 145e5d8

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

This PR includes changesets to release 1 package
Name Type
saleor-dashboard Minor

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

@github-actions github-actions bot temporarily deployed to pr-export-products-test December 7, 2023 09:19 Destroyed
@github-actions github-actions bot temporarily deployed to storybook pr-export-products-test December 7, 2023 09:20 Destroyed
szczecha
szczecha previously approved these changes Dec 7, 2023
andrzejewsky
andrzejewsky previously approved these changes Dec 7, 2023
poulch
poulch previously approved these changes Dec 7, 2023
async () => {
let userEmails: any[] = await this.getEmailsForUser(userEmail);
for (const email of userEmails) {
await console.log(email);
Copy link
Member

Choose a reason for hiding this comment

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

question: do you need this console log here

}

async gotoExistingProductPage(productId: string) {
await console.log(
Copy link
Member

Choose a reason for hiding this comment

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

question: console.log is not async method, why did you put await before

@wojteknowacki wojteknowacki dismissed stale reviews from poulch, andrzejewsky, and szczecha via dc5d1e4 December 8, 2023 14:23
@github-actions github-actions bot temporarily deployed to pr-export-products-test December 8, 2023 14:24 Destroyed
@github-actions github-actions bot temporarily deployed to storybook pr-export-products-test December 8, 2023 14:24 Destroyed
@github-actions github-actions bot temporarily deployed to pr-export-products-test December 8, 2023 14:37 Destroyed
@github-actions github-actions bot temporarily deployed to storybook pr-export-products-test December 8, 2023 14:37 Destroyed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

4 participants