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

New package for inclusion - meteor-desktop #39

Closed
5 tasks done
KoenLav opened this issue Oct 23, 2019 · 19 comments
Closed
5 tasks done

New package for inclusion - meteor-desktop #39

KoenLav opened this issue Oct 23, 2019 · 19 comments

Comments

@KoenLav
Copy link

KoenLav commented Oct 23, 2019

Build desktop apps with Meteor & Electron. Full integration with hot code push implementation.

Links

Current status of the project

  • Abandoned?

Reasoning

This integration enables building desktop applications for Meteor, just like the Cordova integration enables building mobile applications.

The package combines pre-existing solutions for packaging Electron together with web frameworks. It is loosely coupled with Meteor, modular and well written, so it should be easy to maintain.

We are using this project in production on a large scale and are willing to put in the time to maintain this package. Judging from the open pull requests I think there will be others interested as well.

Progress

  • Discussion finished
  • Moved / Forked
  • Team Created
  • Meteor Org Created and set as maintainer
  • communitypackages org added as a maintainer on Atmosphere
@StorytellerCZ
Copy link
Member

I really like the idea, but prefer to transfer this to core Meteor. 🤣
Let's postpone this till we have a better idea where Tiny plans to go with Meteor.

@Ajaay
Copy link

Ajaay commented Nov 25, 2019

We are using this in a production app also and would be prepared to help out with this.

@jamesgibson14
Copy link

jamesgibson14 commented Nov 26, 2019

I am just looking into using Electron with Meteor, so this package being updated would be great. I will help out as much as I can. It would be nice to see what plans Tiny has in this area.

@StorytellerCZ
Copy link
Member

@wojtkowiak Any official statement?

@stale
Copy link

stale bot commented Aug 16, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Aug 16, 2020
@stale stale bot closed this as completed Aug 23, 2020
@wildhart
Copy link

wildhart commented Sep 8, 2020

I also use meteor-desktop in production so would be happy to help maintain the package.

@KoenLav
Copy link
Author

KoenLav commented Sep 9, 2020

Seems @wojtkowiak has been vacant in the repository since January, while Electron is picking up the pace with breaking changes.

Just created this PR to fix an issue while running with Node.js 14: wojtkowiak/meteor-desktop#285

Maybe we should simply fork the repository, start working on it, and ask @wojtkowiak to transfer the npm package?

@KoenLav
Copy link
Author

KoenLav commented Sep 9, 2020

@StorytellerCZ could this be re-opened?

@copleykj copleykj reopened this Sep 9, 2020
@stale stale bot removed the wontfix This will not be worked on label Sep 9, 2020
@copleykj
Copy link
Member

copleykj commented Sep 9, 2020

Maybe we should simply fork the repository, start working on it, and ask @wojtkowiak to transfer the npm package?

I think this should be fine.

@stale
Copy link

stale bot commented Nov 8, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Nov 8, 2020
@stale stale bot closed this as completed Nov 15, 2020
@harryadel
Copy link
Member

Since @wojtkowiak isn't responding, @KoenLav move your fork under Meteor community packages umbrella and @StorytellerCZ can setup a team that includes you and @Ajaay @jamesgibson14. What do you guys think?

@dd137
Copy link

dd137 commented Nov 23, 2020

Our company also is using meteor-desktop in production and we would be happy to help. This package is key for us and bringing it into Meteor Community would make much sense to us. @KoenLav @StorytellerCZ any update? If this is happening feel free to add me to the team.

@KoenLav
Copy link
Author

KoenLav commented Nov 23, 2020

@ all I'm sorry this took so long, but basically meteor-desktop has been working fine for a long time for quite a few Electron version, so I didn't see the rush.

https://github.com/Meteor-Community-Packages/meteor-desktop

@harryadel setting up a team would be great.

@StorytellerCZ could you setup a team with @Ajaay , @dorandeluz and me?

@jamesgibson14 are you still interested in helping out/working on this?

@darqs would you also like to be included?

@wojtkowiak just another ping to make sure you're aware we're moving ahead with this :)

@StorytellerCZ
Copy link
Member

@KoenLav Done!

@KoenLav
Copy link
Author

KoenLav commented Dec 5, 2020

@StorytellerCZ I'm thinking maybe we should also add @wojtkowiak so when/if he returns to the project he can choose to work on it from within Meteor Community Packages (instead of having to request to be added to the team first)?

@copleykj copleykj reopened this Mar 16, 2021
@stale stale bot removed the wontfix This will not be worked on label Mar 16, 2021
@copleykj
Copy link
Member

Ok, since we haven't had any contact from @wojtkowiak, we will just move forward with a fork. @StorytellerCZ can you set up anything that's necessary with NPM?

@KoenLav do you have any time to allocate to this?

@KoenLav
Copy link
Author

KoenLav commented Mar 20, 2021

I actually already forked the package a while ago and gave the team access:

https://github.com/Meteor-Community-Packages/meteor-desktop

I haven't made any changes yet, because for us (up until Electron 11) it has been working fine (as long as you set enableRemoteModule to true).

Feel free to raise issues in the repository; I guess we should try and work out how we can get this to work with Electron 12.

@copleykj
Copy link
Member

Thanks for the update @KoenLav. I've also invited @a4xrbj1 to join this org and the meteor-desktop team. Hopefully we can get some PR's rolling and release an update in the near future.

I've also created the #meteor-desktop channel on the slack worspace to facilitate any discussion about the package the doesn't fit into an issue.

@copleykj
Copy link
Member

I think this should be in order at this point. If there is anything further that is needed, feel free to contact me in slack or by email.

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

No branches or pull requests

8 participants