Skip to content

Conversation

@sergiopvilar
Copy link

Closes #65 .

@manosim manosim added the enhancement New feature or enhancement to existing functionality label Jul 24, 2015
@manosim
Copy link
Member

manosim commented Jul 24, 2015

That looks really nice @sergiovilar ! Will give it a try ASAP and let you know how it goes! Can you also fix the test coverage? If not, I can give it a go!

@sergiopvilar
Copy link
Author

Hi @ekonstantinidis, thanks! I fixed in the last commit, it's all passing as you can see here.

@manosim
Copy link
Member

manosim commented Jul 24, 2015

While it seems to work (it does open when the mac starts), it seems that when you toggle the setting("Open at start up") the UI crashes. To reproduce just toggle that setting and then press the back button - for me it doesn't go back. Will have a further look too!

@sergiopvilar
Copy link
Author

@ekonstantinidis I'll take a look!

@sergiopvilar
Copy link
Author

@ekonstantinidis now seems to be working fine! Please take a look 😄

@manosim
Copy link
Member

manosim commented Jul 25, 2015

@sergiovilar Works niiice! Merged & thanks! 🚀 🚀 🚀

manosim pushed a commit that referenced this pull request Jul 25, 2015
Added "Open at startup" option
@manosim manosim merged commit 37ad3c5 into gitify-app:master Jul 25, 2015
@sergiopvilar
Copy link
Author

😁

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

Labels

enhancement New feature or enhancement to existing functionality

Development

Successfully merging this pull request may close these issues.

2 participants