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

Implement a full-usable frontend #8

Closed
dArignac opened this issue Oct 2, 2013 · 6 comments
Closed

Implement a full-usable frontend #8

dArignac opened this issue Oct 2, 2013 · 6 comments
Assignees
Milestone

Comments

@dArignac
Copy link
Member

dArignac commented Oct 2, 2013

Please do so ;-)

@ghost ghost assigned mmrose Oct 2, 2013
dArignac pushed a commit that referenced this issue May 21, 2014
 added sceleton view and url for subscription deletion
 #8
@dArignac
Copy link
Member Author

To sum it up, that means the following things:

  • add/edit/delete email notifications
  • add/edit/delete subscription for one or more ASINs with each an email notification and a price

@mmrose
Copy link
Member

mmrose commented May 7, 2015

Angular-Frontend is near a usable state. At the moment subscriptions on product detail page are not filtered by user.

dArignac added a commit that referenced this issue May 8, 2015
…not limited in any way) and as we'll change to the new frontend soon, the graphs are disabled for now #8
dArignac added a commit that referenced this issue May 8, 2015
@dArignac
Copy link
Member Author

dArignac commented May 8, 2015

One more thing... ;)
The image urls are HTTP by default. But we have a setting PRICE_MONITOR_IMAGES_USE_SSL to enforce them via HTTPS. I can't remember how we solved this in the "old" master, there is this Product.__get_image_url method that changes the image host to HTTPS if configured but it was nowhere used. I added a method Product.get_image_urls to get all image URLs with correct schema for use in the old frontend, see f247f54. This has to be adapted for the angular frontend somehow.

@mmrose
Copy link
Member

mmrose commented May 9, 2015

It uses the properties again as this plays nicely together with serialization of DRF. I'm looking into that.

@mmrose
Copy link
Member

mmrose commented Jun 6, 2015

I would close this, as it is implemented and the rest should be done via new features or bugs, do you agree @dArignac ?

@dArignac
Copy link
Member Author

dArignac commented Jun 6, 2015

Agreed.

@mmrose mmrose closed this as completed Jun 6, 2015
@mmrose mmrose removed the in progress label Jun 6, 2015
@dArignac dArignac added this to the v0.3b1 milestone Sep 20, 2015
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

2 participants