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

Disable checkbox for products that are not available in selected channels #4841

Merged
merged 4 commits into from
May 14, 2024

Conversation

Cloud11PL
Copy link
Member

@Cloud11PL Cloud11PL commented May 7, 2024

What type of PR is this?

  • 💅 Refactor
  • 🌟 Feature
  • 🔥 Bug Fix
  • 🔩 Maintenance
  • 🛠 Workflow CI/CD changes

Related Issues or Documents

  • closes #

Usage Instructions, Screenshots, Recordings

CleanShot 2024-05-07 at 12 05 04

Have you written tests?

  • Yes!
  • No... here is why: Writing tests are mandatory, please replace this text with why test are not included in this PR

[Optional] Description

Copy link

changeset-bot bot commented May 7, 2024

🦋 Changeset detected

Latest commit: e4ab56c

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 Patch

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-4841 May 7, 2024 10:06 Destroyed
@github-actions github-actions bot temporarily deployed to pr-4841 May 7, 2024 10:24 Destroyed
@Cloud11PL Cloud11PL requested a review from a team May 7, 2024 10:24
@Cloud11PL Cloud11PL marked this pull request as ready for review May 7, 2024 10:24
@Cloud11PL Cloud11PL requested a review from a team as a code owner May 7, 2024 10:24
@Cloud11PL Cloud11PL requested review from witoszekdev and lkostrowski and removed request for a team May 7, 2024 10:24
Copy link
Member

@poulch poulch left a comment

Choose a reason for hiding this comment

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

Check this PR #4826, you could add your solution to my branch

@Cloud11PL
Copy link
Member Author

@poulch this branch will be merged to the one you mentioned 😄

@yellowee yellowee self-requested a review May 8, 2024 06:44
Copy link
Member

@poulch poulch left a comment

Choose a reason for hiding this comment

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

shouldn't you add text below each row like I did?
Product unavailable in shipping method channels

@andrzejewsky andrzejewsky self-requested a review May 10, 2024 09:40
Copy link
Member

@andrzejewsky andrzejewsky left a comment

Choose a reason for hiding this comment

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

Hey @Cloud11PL please review @poulch pull request because seems like he did something very similar or even exact as you here. While code can be different, we should keep the same consistency between those UI

@github-actions github-actions bot temporarily deployed to pr-4841 May 10, 2024 10:08 Destroyed
@Cloud11PL Cloud11PL requested a review from poulch May 13, 2024 08:55
@Cloud11PL Cloud11PL merged commit dc11a69 into merx-189-voucher-channels May 14, 2024
15 checks passed
@Cloud11PL Cloud11PL deleted the MERX-383-assign-products-fix branch May 14, 2024 08:24
Cloud11PL added a commit that referenced this pull request May 17, 2024
#4826)

* Get product search channel listing

* Show is product incuded in selected channels

* Disable checkbox and show info when no channels overlap

* Add changeset

* Make solution more general

* Add selected channels to other dialogs

* Update fixture

* Update changset

* Fix types

* Update mocks

* Extract messages

* Update messages

* Disable checkbox for products that are not available in selected channels (#4841)

* check if product is available in channel

* add changesets

* show message why product's disabled

* extract messages

* Unify assign product modals (#4864)

---------

Co-authored-by: Wojciech Mista <[email protected]>
karola312 pushed a commit that referenced this pull request May 23, 2024
#4826)

* Get product search channel listing

* Show is product incuded in selected channels

* Disable checkbox and show info when no channels overlap

* Add changeset

* Make solution more general

* Add selected channels to other dialogs

* Update fixture

* Update changset

* Fix types

* Update mocks

* Extract messages

* Update messages

* Disable checkbox for products that are not available in selected channels (#4841)

* check if product is available in channel

* add changesets

* show message why product's disabled

* extract messages

* Unify assign product modals (#4864)

---------

Co-authored-by: Wojciech Mista <[email protected]>
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.

4 participants