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

[Abandoned Plugin] M73 ETA Override #619

Closed
adelyser opened this issue Sep 18, 2020 · 18 comments · Fixed by #705
Closed

[Abandoned Plugin] M73 ETA Override #619

adelyser opened this issue Sep 18, 2020 · 18 comments · Fixed by #705

Comments

@adelyser
Copy link

Which Plugin? (URL to plugin page on plugin repository)
sysadminsh/OctoPrint-M73ETAOverride#18

Ticket on plugin's issue tracker confirming abandonment:
[Feature Request] Python 3 Compatibility #18

Is there someone open to adopting it? ("no" or @-mention user):
no

@ShanDestromp
Copy link

Here; includes the Python3 fix and I even went ahead and added #17 This should be considered temporary, I only know enough python to be dangerous and active development is beyond my talents. Basically someone should take this over ASAP before I hurt myself.

That said I installed via zip and everything seems to be working as it should; although plugin manager didn't enable it by default on install (might have been because I'd disabled the old one and it saved that state).

@cp2004
Copy link
Member

cp2004 commented Sep 19, 2020

There is also OctoPrint Prusa M73 Override, repository listing that I updated (Py 3) & tested a while back, it looks like it does the same thing? If so, there may not be a need to find a new maintainer. The author (kanocz) on that repository is responsive to PRs but not actively developing the plugin.

@ghost
Copy link

ghost commented Sep 22, 2020

Is there someone open to adopting it? ("no" or @-mention user):
@Moorelife (a.k.a. I) am into delving into this plugin. I recently got into 3D printing and using OctoPrint, finding both very interesting. As a Test Automation engineer I have sufficient knowledge of both Python 3 and 2. Feel free to speak up if you think I shouldn't take this task in hand.... :-)

@ghost
Copy link

ghost commented Sep 23, 2020

Here; includes the Python3 fix and I even went ahead and added #17 This should be considered temporary, I only know enough python to be dangerous and active development is beyond my talents. Basically someone should take this over ASAP before I hurt myself.

@ShanDestromp consider this taken over. :-) The adopted code can be found here. Thanks for doing the quick fix, that will give me some time to delve into the Octoprint plugin tutorial and the code, before actually getting into de active development part.

@ShanDestromp
Copy link

@foosel @adelyser That's one you can scratch off the abandoned list 👍

@Moorelife Appreciate the take up. I just happened to be looking at the repo and saw the work was already done.

@cp2004
Copy link
Member

cp2004 commented Sep 23, 2020

@Moorelife - Does this mean you will be able to take over the repository listing straight away (ie. new users install your plugins, old ones get told to migrate), or you would like a little more time?

When you want to officially take over, feel free to fork this repository and edit the plugin's listing to point to your fork, and anything else you want changed. Then send in a PR back here and it can be official!

@ghost
Copy link

ghost commented Sep 23, 2020 via email

@ghost
Copy link

ghost commented Sep 23, 2020

Hmm, although the plugin tutorial could do with a few small updates, it was enough to see plugin development is quite straightforward. Now delving into the code of the M73ETA plugin itself...

@gdombiak
Copy link
Contributor

I was going to offer taking it but happy to see @Moorelife already took it! Thanks André

@richardsondev
Copy link

Hey there, has there been any update to this?
Best regards

@ghost
Copy link

ghost commented Dec 3, 2020 via email

@gdombiak
Copy link
Contributor

gdombiak commented Dec 3, 2020

I would be happy to take ownership on this plugin if no one took it already!

Gaston

@ghost
Copy link

ghost commented Dec 3, 2020 via email

@gdombiak
Copy link
Contributor

gdombiak commented Dec 4, 2020

Hi @Moorelife, do you happen to know the process for taking ownership? I guess I will fork the project, update existing entry in https://github.com/OctoPrint/plugins.octoprint.org? How do I mark it as no longer abandoned?

Thanks,
Gaston

@jneilliii
Copy link
Contributor

We can handle that part once you have the fork ready and have updated the update check hook to point to your repo and release a new version on it. Make sure to update the version number in setup.py before creating the release.

@jneilliii
Copy link
Contributor

but yes, you'll need to create a PR against this plugin's repo with author/authors updated and updated URLs to your fork once you have the release ready. It's similar to what you would do if you were registering a new plugin as described here. Please just make sure to do it from a separate branch on your fork of plugins.octoprint.org.

@gdombiak
Copy link
Contributor

gdombiak commented Dec 5, 2020

Thanks @jneilliii for the info. Will work on this during the weekend. Trying to finish Siri control of Enclosure plugin

@gdombiak
Copy link
Contributor

gdombiak commented Dec 5, 2020

All done. Made the plugin compatible with Python 3. Updated all information plus improved description and setup. PR sent.

jneilliii added a commit to jneilliii/plugins.octoprint.org that referenced this issue Dec 8, 2020
jneilliii added a commit that referenced this issue Dec 8, 2020
Add necessary notices for plugin adoptions. finalizes #699, closes #619
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants