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

[FEATURE] re-design labels #552

Closed
Magikarplvl4 opened this issue Sep 24, 2016 · 8 comments
Closed

[FEATURE] re-design labels #552

Magikarplvl4 opened this issue Sep 24, 2016 · 8 comments

Comments

@Magikarplvl4
Copy link
Contributor

Magikarplvl4 commented Sep 24, 2016

The labels witch we show now is: Available, Not Available, Requested.

If i can make a suggestion, will it be better to make labels like:

  1. Available on plex [Green] - showing after the first episode is available on plex
  2. Processing request [light blue] - showing after the request is approved or the first episode is not available yet on plex
  3. Pending approval [yellow] - showing after the request and the request is not approved yet by the admin
  4. Not Requested yet [Orange] - Showing default if the tv show isn't on plex and requested
@ProtoJazz
Copy link
Contributor

I didnt know PlexRequest was .net. I can take a look at this

@tidusjar
Copy link
Member

tidusjar commented Sep 30, 2016

It is! This feature is mainly all in HTML and Handlebars template changes :)
@ProtoJazz

@ProtoJazz
Copy link
Contributor

Awesome. Ill start on it this weekend for my first hacktoberfest project

@tidusjar
Copy link
Member

Awesome!

On Fri, Sep 30, 2016 at 8:14 PM Jim MacKenize notifications@github.com
wrote:

Awesome. Ill start on it this weekend for my first hacktoberfest project


You are receiving this because you commented.

Reply to this email directly, view it on GitHub
#552 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVaLJG2EaH98EVMPJe-edszPCZwm49oks5qvV-GgaJpZM4KFxwC
.

Thanks,
Jamie

@Magikarplvl4
Copy link
Contributor Author

Thanks! Cool

@ProtoJazz
Copy link
Contributor

Checked out the project tonight and took a shot at this. Currently only for TV shows, wasn't sure if this was wanted for movies as well. Not a lot of actual work, but took some learning and getting familiar with how it works. Im also pretty new to the asp part of .net, but I don't think much of that applied to this.

Hadn't used handlebars before, so it might not be the best, but it seems to work.

#562

@Magikarplvl4
Copy link
Contributor Author

@ProtoJazz Thanks for you great work.
@tidusjar where is the waiting for release label? :(

@tidusjar
Copy link
Member

tidusjar commented Oct 8, 2016

My bad!

Might release tomorrrow if I can get some stuff done tonight.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants