Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Update to newest Teslamate Release + Small fix in Grafana config #127

Merged
merged 4 commits into from
Feb 7, 2024

Conversation

vh2002
Copy link
Contributor

@vh2002 vh2002 commented Feb 6, 2024

No active testing done, assuming just an realease update is enough when checking the original changelog.

Closes #125 as it is fixed in the original repo.

Raised errors when installing
Update to newest Teslamate Release
Update to newest Teslamate Release
Update to newest Teslamate Release
@matt-FFFFFF
Copy link
Owner

PR branch does not have access to secrets

Thank you for this PR. I will pull into a local branch and run

@vh2002
Copy link
Contributor Author

vh2002 commented Feb 7, 2024

One question regarding the PRs. As the secrets are needed to run the checks successful, what is the point of even having PRs and is there a potential way around?

@shivaagarwal1
Copy link

PR branch does not have access to secrets

Thank you for this PR. I will pull into a local branch and run

Hi Matt. Apologies for pushing it, but do you know when you will be able to update the add-on? I am checking the page every 5 mins like a moron :p

@matt-FFFFFF matt-FFFFFF merged commit a205118 into matt-FFFFFF:main Feb 7, 2024
0 of 4 checks passed
@matt-FFFFFF
Copy link
Owner

One question regarding the PRs. As the secrets are needed to run the checks successful, what is the point of even having PRs and is there a potential way around?

Because I haven't skipped the docker login & push to docker hub steps for PRs from a fork. This is all possible of course but it's just time.

At the beginning it was just me maintaining this so it wasn't an issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TeslaApi.Error / %{"error" => "Endpoint is only available on fleetapi
3 participants