Skip to content

add shipping estimate as variable#8

Merged
wagnerlduarte merged 1 commit intomasterfrom
feature/BRPA-352-sort-by-expression-estimate
Sep 29, 2021
Merged

add shipping estimate as variable#8
wagnerlduarte merged 1 commit intomasterfrom
feature/BRPA-352-sort-by-expression-estimate

Conversation

@wagnerlduarte
Copy link
Collaborator

What problem is this solving?

Added minShippingPrice, maxShippingPrice, minShippingEstimate and maxShippingEstimate variables to calculate math expression to sort sellers.

How to test it?

Workspace

On expression field put a value like: ((productPrice + minShippingPrice) * minShippingEstimate)

Screenshots or example usage:

image

How does this PR make you feel? 🔗

@wagnerlduarte wagnerlduarte self-assigned this Sep 28, 2021
@vtex-io-ci-cd
Copy link

vtex-io-ci-cd bot commented Sep 28, 2021

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

vtex-io-docs-bot bot commented Sep 28, 2021

Beep boop 🤖

I noticed you didn't make any changes at the docs/ folder

  • There's nothing new to document 🤔
  • I'll do it later 😞

In order to keep track, I'll create an issue if you decide now is not a good time

  • I just updated 🎉🎉

@github-actions
Copy link

Fails
🚫

👥 There are no reviewers assigned to this pull request!

🚫

We follow keepachangelog standards.

Please, change the file CHANGELOG.md adding a small summary of your changes under the [Unreleased] section. Example:

 ## [Unreleased]

+ ### Fixed
+ - Issue with image sizing in the SKU Selector.

Possible types of changes:

  • Added for new features.
  • Changed for changes in existing functionality.
  • Deprecated for soon-to-be removed features.
  • Removed for now removed features.
  • Fixed for any bug fixes.
  • Security in case of vulnerabilities.

Generated by 🚫 dangerJS against b43abb3

Base automatically changed from feature/BRPA-352-sort-by-expression to master September 28, 2021 20:57
@vtex-io-docs-bot
Copy link

Beep boop 🤖 That's ok, I created an issue for this so we don't forget

@wagnerlduarte wagnerlduarte deleted the feature/BRPA-352-sort-by-expression-estimate branch September 29, 2021 17:01
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.

2 participants