Skip to content
This repository has been archived by the owner on Jul 24, 2020. It is now read-only.

Add all FSK 18 articles to notification if region is DE #92

Open
dArignac opened this issue Jan 16, 2016 · 2 comments
Open

Add all FSK 18 articles to notification if region is DE #92

dArignac opened this issue Jan 16, 2016 · 2 comments
Assignees
Milestone

Comments

@dArignac
Copy link
Member

This is a redefinition of #19 .

If the current set api region is DE and a product hits the price limit, and the product is a FSK 18 article, then also include all other subscribed FSK 18 products within the notification mail. These products shall be sorted, starting with the lowest price.

As we send text mails, this could look like this (just a suggestion, we'll see how this looks in a real mail):

The price limit of <PRICE-LIMIT> has been reached for the article "<PRODUCT>"
Current price is at<PRICE> (<DATE-OF-PRICE>).

Please support our platform by using this affiliate link for buying the product: http://www.amazon.de/dp/<ASIN>/?tag=<ASSOC-TAG>
Adjust the price limits for the products here: <DEEPLINK>

As this is a FSK 18 article, here are your other subscribed FSK 18 articles:

<PRODUCT>
<PRICE> (<DATE-OF-PRICE>)
<REFLINK>
<DEEPLINK>

<PRODUCT>
<PRICE> (<DATE-OF-PRICE>)
<REFLINK>
<DEEPLINK>

Regards,
The Team
@dArignac dArignac self-assigned this Jan 16, 2016
@dArignac dArignac added this to the v0.7 milestone Jan 16, 2016
@dArignac dArignac mentioned this issue Jan 16, 2016
@dArignac dArignac changed the title Add all FSK 18 articles to notification if location is Germany Add all FSK 18 articles to notification if region is DE Jan 16, 2016
dArignac added a commit that referenced this issue Jan 16, 2016
dArignac added a commit that referenced this issue Jan 16, 2016
dArignac added a commit that referenced this issue Jan 16, 2016
[ci skip]
@dArignac
Copy link
Member Author

Hm, this does not work when using stmp mailing, somehow gets stuck without exception and blocks everything. Need to rework.

@dArignac dArignac reopened this Jan 16, 2016
@dArignac
Copy link
Member Author

Additionaly, "Ages 18 and over" was not considered.

dArignac added a commit that referenced this issue Jan 20, 2016
@dArignac dArignac modified the milestones: v0.8, v0.7 Apr 4, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant