Skip to content
This repository has been archived by the owner on Jan 23, 2024. It is now read-only.

Add summary of playlist rules into playlist description #19

Merged
merged 1 commit into from
Dec 7, 2015

Conversation

bennettmcelwee
Copy link
Contributor

This commit adds a summary of the playlist rule into the description, so it appears as something like this:

Automatically managed by Autoplaylists for Google Music™ (durationMillis > 180000 and durationMillis < 240000 and (title matches blue or title matches green or artist = Kwyjibo)) sort by title descending

I find it useful to have this available, so I can keep track of my many playlists. Maybe others will find it useful too.

BTW I love this extension. Thanks for writing it -- It fills a glaring hole in Google Music.

@simon-weber
Copy link
Owner

Awesome; thanks.

I'm thinking of charging for the extension at some point and letting contributors use it for free. Is that something you're comfortable with? I don't have a contributor agreement or anything yet since I wasn't expecting contributions so soon, but this is the general idea: #10 (comment).

@bennettmcelwee
Copy link
Contributor Author

Hi Simon,

Sorry, I didn't actually read CONTRIBUTING.md before sending my PR. I
actually just made that change for myself, but then thought it might be
useful for others too.

Yes, I'm happy to contribute even if you're planning to charge for the
extension.

Cheerio,
Bennett.

Bennett McElwee
bennett@thunderguy.com

On 5/12/15 5:35 am, Simon Weber wrote:

Awesome; thanks.

I'm thinking of charging for the extension at some point and letting
contributors use it for free. Is that something you're comfortable
with? I don't have a contributor agreement or anything yet since I
wasn't expecting contributions so soon, but this is the general idea:
#10 (comment)
#10 (comment).


Reply to this email directly or view it on GitHub
#19 (comment).

@simon-weber
Copy link
Owner

I made that change for myself, but then thought it might be useful for others too.

Exactly what I was hoping for when open sourcing this =) Thanks!

simon-weber added a commit that referenced this pull request Dec 7, 2015
Add summary of playlist rules into playlist description
@simon-weber simon-weber merged commit bd1fa9c into simon-weber:master Dec 7, 2015
simon-weber added a commit that referenced this pull request Dec 10, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants