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

Enhanced rating support with Allocine/Filmstarts/Screenrush #72

Open
Arnaud-G1 opened this issue Aug 28, 2014 · 0 comments
Open

Enhanced rating support with Allocine/Filmstarts/Screenrush #72

Arnaud-G1 opened this issue Aug 28, 2014 · 0 comments
Labels
bug Bugs and dealbrakers, big or small. Needs to be cleared before releasing. enhancement Enhancements and improvements, nice to haves for a next release but never mandatory.
Milestone

Comments

@Arnaud-G1
Copy link

Hi,

I'm currently looking to improve the rating support.

My reference engine is allocine and this engine provide two ratings (user rating and press rating) which are very useful
I would like to show both on list (read only), show (read only) and edit (RW) pages.
And next to that I would like to be able to sort entries in list page based on either one or the other rating.

Next to it, Allocine can also provide press rating from various newspapers.
In that perspective I'm enhancing allocine.php to use directly Allocine API.

Now my approach for implementation:

  • Update allocine.php (based on Etienne Gauvin API)
  • Dual rating: Easy step is to add a new table entry for press rating (ok if only 2 ratings)
    Otherwise I need to make a new table with all ratings per movie
  • Adapt filter stuff on index.php for the ordering options

I'm currently only using elegant template, but I'll adapt the other as well when get it stable..
Support of other variants like filmstart.de or screenrush,co.uk is also planned later on as it relies on same allocine API.

Any advise is welcome

I'm new to Git so I didn't release my files yet.

Arnaud

@johanneskonst johanneskonst added enhancement Enhancements and improvements, nice to haves for a next release but never mandatory. bug Bugs and dealbrakers, big or small. Needs to be cleared before releasing. labels Dec 8, 2022
@johanneskonst johanneskonst mentioned this issue Dec 8, 2022
@johanneskonst johanneskonst added this to the 4.2 milestone Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugs and dealbrakers, big or small. Needs to be cleared before releasing. enhancement Enhancements and improvements, nice to haves for a next release but never mandatory.
Projects
None yet
Development

No branches or pull requests

2 participants