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

Updated OctoLight API information #1310

Merged
merged 4 commits into from
Jul 15, 2024

Conversation

thomst08
Copy link
Contributor

  • Updated OctoLight API information
  • Added notification regarding older version of OctoLight as it changes the API

Please let me know if this is not a good use of the notices file as I am OK if it shouldn't be used in this way.
Thank you.

- Updated OctoLight API information
- Added notification regarding older version of OctoLight as it changes the API
- Missing spaces caused a build error?
@cp2004
Copy link
Member

cp2004 commented Jul 10, 2024

Would it be easier to keep the API documentation just on your plugin's GitHub readme, and just provide a link from the plugin repository? That should save you making lots of PRs here if you want to change the docs (even small corrections you might find) in the future?

@thomst08
Copy link
Contributor Author

Honestly, I don't see it changing much beyond this but you have a point, if you would prefer I do that then let me know and I will change the file straight way.

- Removed API info
- Setup link to GH page
- Updated link to cleaner link
@thomst08
Copy link
Contributor Author

@cp2004, thanks for the suggestion, thinking about it, I think you have a point. The change is done.

@jneilliii
Copy link
Contributor

hmm...we've never had plugin notices outside of plugin adoption ones before. Need @foosel input on this one.

- Removed added notification
@thomst08
Copy link
Contributor Author

Sorry for the delay, I removed the notification addition.
Sorry for causing any issues.

@jneilliii jneilliii merged commit 959a4c1 into OctoPrint:gh-pages Jul 15, 2024
4 checks passed
@thomst08 thomst08 deleted the octolight-changes-24.07.10 branch July 15, 2024 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants