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

Feature/salesChannel #66

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

germanBonacchi
Copy link

What problem is this solving?

Change condition-layout.binding to be able to have the salesChannel as subject

How to test it?

Link and store-theme with a vtex.condition-layout@2.x:condition-layout.binding block with condition like this "conditions": [ { "subject": "salesChannel", "arguments": { "id": "2" } } ],
Workspace

Screenshots or example usage:

Captura desde 2022-10-11 11-39-59
Captura desde 2022-10-11 11-16-46
Captura desde 2022-10-11 11-43-58
Captura desde 2022-10-11 11-37-36

@germanBonacchi germanBonacchi requested review from a team as code owners October 11, 2022 14:45
@germanBonacchi germanBonacchi requested review from kaio-donadelli and removed request for a team October 11, 2022 14:45
@vtex-io-ci-cd
Copy link

vtex-io-ci-cd bot commented Oct 11, 2022

Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖

Please select which version do you want to release:

  • Patch (backwards-compatible bug fixes)

  • Minor (backwards-compatible functionality)

  • Major (incompatible API changes)

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

  • No thanks, I would rather do it manually 😞

@vtex-io-docs-bot
Copy link

Beep boop 🤖

Thank you so much for keeping our documentation up-to-date ❤️

@germanBonacchi germanBonacchi requested review from filipewl, tlgimenes and hellofanny and removed request for a team October 11, 2022 14:45
@sheilagomes
Copy link

@germanBonacchi , when you need translation/reviews please open a JIRA task for the Product Localization team. I opened one for this PR, but here are the instructions for opening one next time.

@germanBonacchi
Copy link
Author

@sheilagomes i can't access to the links that you share, i am not from vtex anymore

@sheilagomes
Copy link

@sheilagomes i can't access to the links that you share, i am not from vtex anymore

Oh, ok, no problem! The task is open and the final delivery date is October 21. When it's done, I'll push the corrections to the PR.

@germanBonacchi
Copy link
Author

@sheilagomes I can´t use the VTEX IO CI/CD Bot to select a versions, but it needs to be a minor

@sheilagomes
Copy link

@germanBonacchi , we need to clarify the term Sales Channel used in this app, as sometimes this term is used as equivalent to Trade Policy in other apps. What do you mean exactly by sales channel here? Can you give an example of a sales channel, just for us to be sure on the translation?

@germanBonacchi
Copy link
Author

germanBonacchi commented Oct 21, 2022

Hi @sheilagomes , yes is a confused term sometimes.

I'm talking about Sales Channel equivalent to Trade Policy.

For example we have 4 createad here: https://beautycounterqa.myvtex.com/admin/Site/Store.aspx

And if the user is in the sc=1 we want to show some content and if sc=2 show other.

image

Comment on lines +24 to +27
"admin/editor.condition-layout-binding.condition": "Condition Binding block",
"admin/editor.condition-layout-binding.conditions": "Conditions Binding",
"admin/editor.condition-layout-binding.subject.bindingId": "Selected bindingId",
"admin/editor.condition-layout-binding.subject.salesChannel": "Selected Sales Channel"

Choose a reason for hiding this comment

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

Suggested change
"admin/editor.condition-layout-binding.condition": "Condition Binding block",
"admin/editor.condition-layout-binding.conditions": "Conditions Binding",
"admin/editor.condition-layout-binding.subject.bindingId": "Selected bindingId",
"admin/editor.condition-layout-binding.subject.salesChannel": "Selected Sales Channel"
"admin/editor.condition-layout-binding.condition": "Binding conditions block",
"admin/editor.condition-layout-binding.conditions": "Binding conditions",
"admin/editor.condition-layout-binding.subject.bindingId": "Selected binding ID",
"admin/editor.condition-layout-binding.subject.salesChannel": "Selected trade policy"

Comment on lines +24 to +27
"admin/editor.condition-layout-binding.condition": "Condition Binding block",
"admin/editor.condition-layout-binding.conditions": "Conditions Binding",
"admin/editor.condition-layout-binding.subject.bindingId": "Selected bindingId",
"admin/editor.condition-layout-binding.subject.salesChannel": "Selected Sales Channel"

Choose a reason for hiding this comment

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

Suggested change
"admin/editor.condition-layout-binding.condition": "Condition Binding block",
"admin/editor.condition-layout-binding.conditions": "Conditions Binding",
"admin/editor.condition-layout-binding.subject.bindingId": "Selected bindingId",
"admin/editor.condition-layout-binding.subject.salesChannel": "Selected Sales Channel"
"admin/editor.condition-layout-binding.condition": "Binding conditions block",
"admin/editor.condition-layout-binding.conditions": "Binding conditions",
"admin/editor.condition-layout-binding.subject.bindingId": "Selected binding ID",
"admin/editor.condition-layout-binding.subject.salesChannel": "Selected trade policy"

Comment on lines +24 to +27
"admin/editor.condition-layout-binding.condition": "Bloque de condición Binding",
"admin/editor.condition-layout-binding.conditions": "Condiciones Binding",
"admin/editor.condition-layout-binding.subject.bindingId": "BindingId seleccionado",
"admin/editor.condition-layout-binding.subject.salesChannel": "Sales Channel seleccionado"

Choose a reason for hiding this comment

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

Suggested change
"admin/editor.condition-layout-binding.condition": "Bloque de condición Binding",
"admin/editor.condition-layout-binding.conditions": "Condiciones Binding",
"admin/editor.condition-layout-binding.subject.bindingId": "BindingId seleccionado",
"admin/editor.condition-layout-binding.subject.salesChannel": "Sales Channel seleccionado"
"admin/editor.condition-layout-binding.condition": "Bloque de condiciones de vínculo",
"admin/editor.condition-layout-binding.conditions": "Condiciones de vínculo",
"admin/editor.condition-layout-binding.subject.bindingId": "ID del vínculo seleccionado",
"admin/editor.condition-layout-binding.subject.salesChannel": "Política comercial seleccionada"

Comment on lines +24 to +27
"admin/editor.condition-layout-binding.condition": "Bloco de condições Binding",
"admin/editor.condition-layout-binding.conditions": "Condições Binding",
"admin/editor.condition-layout-binding.subject.bindingId": "BindingId selecionado",
"admin/editor.condition-layout-binding.subject.salesChannel": "Sales Channel selecionado"

Choose a reason for hiding this comment

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

Suggested change
"admin/editor.condition-layout-binding.condition": "Bloco de condições Binding",
"admin/editor.condition-layout-binding.conditions": "Condições Binding",
"admin/editor.condition-layout-binding.subject.bindingId": "BindingId selecionado",
"admin/editor.condition-layout-binding.subject.salesChannel": "Sales Channel selecionado"
"admin/editor.condition-layout-binding.condition": "Bloco de condições de vínculo",
"admin/editor.condition-layout-binding.conditions": "Condições de vínculo",
"admin/editor.condition-layout-binding.subject.bindingId": "ID do vínculo selecionado",
"admin/editor.condition-layout-binding.subject.salesChannel": "Política comercial selecionada"

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.

None yet

2 participants