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

Merge my Japan NissanConnect implementation? #32

Open
developerfromjokela opened this issue Jul 2, 2024 · 3 comments
Open

Merge my Japan NissanConnect implementation? #32

developerfromjokela opened this issue Jul 2, 2024 · 3 comments
Assignees

Comments

@developerfromjokela
Copy link

Hi.

I've forked your repo and implemented Japanese Domestic Market's NissanConnect app. Here's the repo: https://github.com/developerfromjokela/HomeAssistant-NissanConnect-JP.

I think maybe if we somehow merge it into the main repo so we don't have two identical applications :D

  • DFJ
@dan-r
Copy link
Owner

dan-r commented Jul 3, 2024

I agree it's a good idea to get it merged. Leave it with me and I'll merge your changes.

Happy to make you a contributor for this repo too so you can maintain the Japan side

@dan-r
Copy link
Owner

dan-r commented Jul 3, 2024

@developerfromjokela I've made a start on the changes in the japan branch. Would you mind testing these?

https://github.com/dan-r/HomeAssistant-NissanConnect/tree/japan

@developerfromjokela
Copy link
Author

@developerfromjokela I've made a start on the changes in the japan branch. Would you mind testing these?

https://github.com/dan-r/HomeAssistant-NissanConnect/tree/japan

Hi. I haven't had a chance to test yet. I'll reply back when I can.

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

2 participants