Skip to content

Add sponsoredCount prop#691

Open
hiagolcm wants to merge 2 commits into
masterfrom
feature/sponsoredCount-prop
Open

Add sponsoredCount prop#691
hiagolcm wants to merge 2 commits into
masterfrom
feature/sponsoredCount-prop

Conversation

@hiagolcm

@hiagolcm hiagolcm commented Nov 29, 2024

Copy link
Copy Markdown
Contributor

What problem is this solving?

Currently, the maximum number of sponsoredProducts is hardcoded. This PR allows the developer to change it via store-theme. For example:

"store.search": {
    "blocks": ["search-result-layout"],
    "props": {
      "context": {
        "sponsoredCount": 4
      }
    }
  },

It also removes the sponsoredProductsBehavior which is a deprecated prop.

How to test it?

Workspace

Related to / Depends on

vtex-apps/store#593

@hiagolcm hiagolcm requested review from a team as code owners November 29, 2024 12:58
@hiagolcm hiagolcm requested review from RodrigoTadeuF, camilavcoutinho, gabpaladino and leo-prange-vtex and removed request for a team November 29, 2024 12:58
@vtex-io-ci-cd

vtex-io-ci-cd Bot commented Nov 29, 2024

Copy link
Copy Markdown
Contributor

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
Copy Markdown

Beep boop 🤖

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

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.

3 participants