Skip to content
This repository has been archived by the owner on Jul 14, 2022. It is now read-only.

Use only selection attributes in variant picker #963

Merged

Conversation

orzechdev
Copy link
Contributor

@orzechdev orzechdev commented Dec 8, 2020

I want to merge this change because... it adds usage of only selection attributes in variant picker.

PR intended to be tested with API branch: feature/file-uploads
saleor/saleor#6568

Screenshots

Pull Request Checklist

  1. All visible strings are translated with proper context.
  2. All data-formatting is locale-aware (dates, numbers, and so on).
  3. The changes are tested.
  4. The code is documented (docstrings, project documentation).
  5. Changes are mentioned in the changelog.

Test Environment Config

API_URI=https://feature-file-uploads.api.saleor.rocks/graphql/

@github-actions github-actions bot temporarily deployed to saleor-1357-select-only-selection-variant-attributes December 8, 2020 12:55 Inactive
@github-actions github-actions bot temporarily deployed to storybook saleor-1357-select-only-selection-variant-attributes December 8, 2020 12:55 Inactive
@github-actions github-actions bot temporarily deployed to saleor-1357-select-only-selection-variant-attributes December 8, 2020 13:30 Inactive
@github-actions github-actions bot temporarily deployed to storybook saleor-1357-select-only-selection-variant-attributes December 8, 2020 13:30 Inactive
@patrys
Copy link
Member

patrys commented Dec 8, 2020

@github-actions github-actions bot temporarily deployed to saleor-1357-select-only-selection-variant-attributes December 8, 2020 18:37 Inactive
@github-actions github-actions bot temporarily deployed to storybook saleor-1357-select-only-selection-variant-attributes December 8, 2020 18:37 Inactive
@orzechdev orzechdev marked this pull request as ready for review December 8, 2020 18:47
@krzysztofwolski
Copy link
Member

Cypress will fail because of missing products in test database. Code can be merged

@lgtm-com
Copy link

lgtm-com bot commented Dec 9, 2020

This pull request introduces 2 alerts when merging 8af434b into 6fd7b46 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

@maarcingebala maarcingebala merged commit c7308bc into master Dec 16, 2020
@maarcingebala maarcingebala deleted the SALEOR-1357-select-only-selection-variant-attributes branch December 16, 2020 10:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants