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

Price rules rehaul #157

Open
manycoding opened this issue Aug 27, 2019 · 0 comments
Open

Price rules rehaul #157

manycoding opened this issue Aug 27, 2019 · 0 comments
Labels
Type: Feature New feature or request

Comments

@manycoding
Copy link
Contributor

manycoding commented Aug 27, 2019

Refactor compare_was_now, compare_prices_for_same_urls, compare_names_for_same_urls, compare_prices_for_same_names

compare_was_now:

  • compares numeric value between two job fields
  • Outputs >, <, =

compare_prices_for_same_urls

  • returns url field values difference between 2 jobs (new, same, missing)
  • for same urls, compares price

compare_names_for_same_urls

  • compares names for same between 2 jobs urls

compare_prices_for_same_names

  • returns name field values difference between 2 jobs (new, same, missing)
  • for same names, compares price

The last three do the same:

  • find given field difference - can go in a separate rule
  • compare certain field values between 'same' items. Items are found by a key, different from item_key (e.g. url/name)
@manycoding manycoding added Type: Feature New feature or request Type: Rule and removed Type: Rule labels Aug 27, 2019
@manycoding manycoding changed the title Difference rule - price rules rehaul Price rules rehaul Aug 27, 2019
@manycoding manycoding added this to the 0.3.7 milestone Sep 4, 2019
@manycoding manycoding removed this from the 0.3.7 milestone Oct 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant