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

Fixes the UK Met Office Datahub provider #3497

Closed
wants to merge 7 commits into from
Closed

Fixes the UK Met Office Datahub provider #3497

wants to merge 7 commits into from

Conversation

jargordon
Copy link
Contributor

Fixes #3384

Changed the UKMetOfficeDataHub provider to the new API structure as per the documentation.

API Base URL updated.
Header information amended to the correct key name.
API Secret no longer required so removed.

Changelog updated to reflect the change.

Fixed connections to the new MetOffice data service.
@jargordon jargordon changed the base branch from master to develop July 7, 2024 15:28
@rejas
Copy link
Collaborator

rejas commented Jul 7, 2024

Please base your work upon the develop branch, and not the master branch

@jargordon
Copy link
Contributor Author

Please base your work upon the develop branch, and not the master branch

Have swapped out to the develop branch on the main project. Hopefully this is ok. If i need to re-fork my local copy to the develop rather than master as well then please let me know and i will re-do this.

@sdetweil
Copy link
Collaborator

sdetweil commented Jul 7, 2024

after you git clone, then git checkout develop
then git checkout -b newbranch

and that branch started from develop.

edit, test, fix changelog...
commit to newbranch
push newbranch.up to your fork

then when submit the pr, this newbranch from your fork and select develop on MagicMirror

@khassel
Copy link
Collaborator

khassel commented Jul 7, 2024

Hopefully this is ok.

LGTM

@jargordon
Copy link
Contributor Author

Closing this request due to needing to change to the dev branch.

@jargordon jargordon closed this Jul 7, 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.

Weather module Met Office API Gateway migration
4 participants