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

Prevent multiple reconnects from being scheduled simultaneously. #26

Merged
merged 9 commits into from
May 28, 2022
Merged

Prevent multiple reconnects from being scheduled simultaneously. #26

merged 9 commits into from
May 28, 2022

Conversation

ufodone
Copy link
Owner

@ufodone ufodone commented May 4, 2022

Proposed fix for #25

@ufodone
Copy link
Owner Author

ufodone commented May 27, 2022

@Cinntax Any chance you'd be able to have a look at this?

You previously mentioned maybe handing over maintenance of this project as you no longer have an alarm system that uses it at home so if you're still interested in that, let me know.

@Cinntax Cinntax merged commit 430af20 into ufodone:master May 28, 2022
@Cinntax
Copy link
Collaborator

Cinntax commented May 28, 2022

Hey @ufodone - thanks for checking in with me! Yes I'd definitely still be interested in transferring everything over. I certainly don't mean to slow things up the way I have.

@ufodone
Copy link
Owner Author

ufodone commented May 29, 2022

Thanks, @Cinntax. Let me know how you want to proceed with that.

@ufodone
Copy link
Owner Author

ufodone commented Jun 14, 2022

@Cinntax Hi again! Sorry to bug you again but any thoughts of how to transfer the project? And in the meantime, would you mind pushing a new version so I can get this fix into Home Assistant? Thanks!

@Cinntax
Copy link
Collaborator

Cinntax commented Jun 15, 2022

Hey @ufodone thanks for the reminder. I just pushed version 4.5. I'll make it a priority to ensuring the project is transferred over properly very soon!

@ufodone
Copy link
Owner Author

ufodone commented Jun 18, 2022

Thanks @Cinntax!

@Cinntax
Copy link
Collaborator

Cinntax commented Jun 21, 2022

Hey @ufodone - I was looking into transferring ownership to you. It looks like I can't do that since you have your own copy of it (the fork). I think the easiest thing to do, as long as you have all your code merged in with mine... would be to just delete your fork, and let me know- and I can transfer it over to you. That way you'd have all the PR history, issues, etc.

Also, do you have an account on pypi.org? I can add you as an owner to the pyenvisalink package on there.

@ufodone
Copy link
Owner Author

ufodone commented Jun 23, 2022

Hi @Cinntax, I didn't have any unmerged changes in my fork so I've deleted it now. I've also created an account on pypi.org with the same username as here.

@Cinntax
Copy link
Collaborator

Cinntax commented Jun 23, 2022

Great @ufodone - I have submitted a request to transfer this over to you, and also invited you to the package on pypi.

I forgot to mention- you'll also want to sign up for an account on https://test.pypi.org as well so you can first deploy any new packages there. I can make you an owner there as well as soon as you setup an account.

@ufodone
Copy link
Owner Author

ufodone commented Jun 23, 2022

@Cinntax : Okay, I've accepted the GitHub transfer and pypi invite and also created an account on test.pypi.org. Almost there!

@Cinntax
Copy link
Collaborator

Cinntax commented Jun 23, 2022

Great @ufodone - I also granted you ownership in test.pypi.org. I think we're all set. Thank you!!

@ufodone
Copy link
Owner Author

ufodone commented Jun 26, 2022

Thanks, @Cinntax. Hopefully I'll be able to figure it out from here but I may need to bug you again at some point!

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.

2 participants