Skip to content

version 2.5 has been deprecated, changing default to 3.0#3495

Closed
dkallen78 wants to merge 1 commit into
MagicMirrorOrg:developfrom
dkallen78:master
Closed

version 2.5 has been deprecated, changing default to 3.0#3495
dkallen78 wants to merge 1 commit into
MagicMirrorOrg:developfrom
dkallen78:master

Conversation

@dkallen78

Copy link
Copy Markdown
Contributor

I'm new to using GitHub as a collaborative platform so I'm very new to all of this, let me know if I goofed something and how I can do it better next time!

OpenWeather's One Call API has deprecated version 2.5 so it won't anymore. I changed the default value of apiVersion from 2.5 to 3.0 so newbies like me will have an easier time figuring it out!

@sdetweil

sdetweil commented Jul 4, 2024

Copy link
Copy Markdown
Collaborator

I thought we also had to change the endpoint ?
and all PRs need a CHANGELOG.md update

@dkallen78

Copy link
Copy Markdown
Contributor Author

I'm not sure about changing the endpoint. When I got hourly updates working on my MM, I only had to change the apiVersion property to "3.0" and it worked. My only change was to the default in the module code for the OpenWeather stuff. let me see if I can figure out how to do the changelog stuff...

@khassel

khassel commented Jul 5, 2024

Copy link
Copy Markdown
Collaborator

let me see if I can figure out how to do the changelog stuff...

well, you provided a new PR instead of committing and pushing the changelog stuff to the branch you used in this PR.

So please add the changelog stuff here or the other way: Add the changes done here to the new PR.

We need both in one PR for merging.

@dkallen78

dkallen78 commented Jul 5, 2024 via email

Copy link
Copy Markdown
Contributor Author

@khassel

khassel commented Jul 5, 2024

Copy link
Copy Markdown
Collaborator

workflow:

  • change code, commit, push
  • new changes (e.g. add changelog), add new commit and push
  • create pull request
  • if you want to change something after PR is created, just change code and create new commit and push (you must be on the same branch as before, you used branch master in this PR). The new changes will appear in the existing pull request

You have to do step 4 now.

Let me know if you want to try this yourself (otherwise I can do it).

@khassel

khassel commented Jul 10, 2024

Copy link
Copy Markdown
Collaborator

I added the changes from here to #3496, so closing this.

@khassel khassel closed this Jul 10, 2024
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.

3 participants