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

update dependencies and implement on/off toggle #12

Merged
merged 1 commit into from
Aug 1, 2016

Conversation

tkothe
Copy link
Contributor

@tkothe tkothe commented Jul 24, 2016

update packages (fix #10)

  • update target electron version to 1.26
  • update target electron packager version to 7.2.0
  • update standard to 7.1.2
  • update react version to 0.14.8
  • update menubar to 4.1.2
  • fix correct reference of menu item when packaging app through npm run build

@tkothe tkothe force-pushed the on-off-switch branch 2 times, most recently from f0fb612 to 6dad30b Compare July 24, 2016 12:47
@uiur
Copy link
Owner

uiur commented Jul 25, 2016

Note: Here is a screenshot for me

image


.btn-sendnow {
position: absolute;
left:; 15px;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo? left:;

@uiur
Copy link
Owner

uiur commented Jul 25, 2016

Hi @tkothe. Thanks for your great PR.

I've added a few comments.
btw, it works for Apple Music now, doesn't it? I'm pretty excited.

@tkothe
Copy link
Contributor Author

tkothe commented Jul 27, 2016

@uiureo thanks for the review! I'll do my best to find the time to update based on your comments very soon! (might be a week or two though).

I don't have Apple Music currently (canceled my subscription again recently) but it would indeed be awesome if this worked now!

- bump version to 1.1.1

- upate npm start command to run `electron .`
- remove electron-template-menu dependency

- add toggle to turn automatic sending on/off (fix uiur#11)

update packages (fix uiur#10)
- update target electron version to 1.26
- update target electron packager version to 7.2.0
- update standard to 7.1.2
- update react version to 0.14.8
- update menubar to 4.1.2

- fix correct reference of menu item when packaging app through `npm run build`
@tkothe
Copy link
Contributor Author

tkothe commented Jul 31, 2016

@uiureo, I updated the PR based on your recommendations.

Admittedly, the UI could be nicer showing the label next to the checkbox, but I was unable to get that done in short time - I'm in no way a CSS expert.

@uiur uiur merged commit 208b374 into uiur:master Aug 1, 2016
@uiur
Copy link
Owner

uiur commented Aug 1, 2016

Great work! Thanks @tkothe.

I have a plan to update the UI, so no problem.

@snipe
Copy link

snipe commented May 30, 2018

Hi! I know this issue is a billion years old, but I was wondering if this made it into a release? This is a neat app, and I'd love to use the version that has the on/off toggle, but I'm not seeing that from the version I installed from the releases page.

Is there anything I can help with?

@uiur
Copy link
Owner

uiur commented May 30, 2018

Hi @snipe! Good catch. I've forgot that for a billion years.

I've created the release: https://github.com/uiureo/playing/releases/tag/v1.1.1

Let me know if there's any problem. Thanks!

@snipe
Copy link

snipe commented May 30, 2018

Thank you! Are you still interested in developing this at all? I mean, it does what it says it does, so maybe there’s nothing further to do?

I’m here because you managed to not only solve my iTunes -> slack problem, but your code is pretty great, and I learned a lot about electron apps while pulling your commits from master into a local app so I could compile/use them.

Are you interested in working on this anymore? I’m a backend dev myself, so React/Electron isn’t my jam specifically, but your project inspired me to install XQuartz and wine, so let me know :)

@uiur
Copy link
Owner

uiur commented May 30, 2018

Are you still interested in developing this at all?

No. I haven't used this app now, and don't have any plan to work on this.

But if I had enough time, I would work on:

  • improving UI design
  • windows and linux support
  • form validation
  • updating React / Electron (probably use JSX)

@snipe
Copy link

snipe commented May 30, 2018

I completely understand.

I’ve forked your repo, and if you don’t mind, I may try my hand at tweaking it a bit. I’ll still leave original credits to you of course. It’s a good excuse for me to get my feet wet with Electron, which I’ve been meaning to do forever.

PS - trying to compile this (or anything else) for windows on a Mac is just as awful as you’d imagine.

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

Successfully merging this pull request may close these issues.

on/off switch for automatically sending the currently played song update electron, react and other packages
3 participants