Skip to content

Tp api 7 #32

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Tp api 7 #32

wants to merge 9 commits into from

Conversation

KillerBOSS2019
Copy link
Owner

support for API v7 with all new Deprecation & new features

@KillerBOSS2019 KillerBOSS2019 temporarily deployed to pypi September 29, 2023 22:00 — with GitHub Actions Inactive
@KillerBOSS2019 KillerBOSS2019 temporarily deployed to pypi September 29, 2023 22:13 — with GitHub Actions Inactive
@KillerBOSS2019 KillerBOSS2019 temporarily deployed to pypi September 29, 2023 23:00 — with GitHub Actions Inactive
@KillerBOSS2019 KillerBOSS2019 temporarily deployed to pypi September 30, 2023 05:08 — with GitHub Actions Inactive
@oddbear
Copy link

oddbear commented Mar 3, 2024

Hi.

What is the current state on this as it's not merged in yet?
I am trying to update the C# API to version 7/8, but a lot of the stuff in the documentation seems to not be working.

Do you know if the API 7 is broken at this time? As I guess you would have been testing this a lot. Or if there is any special things I need to do to get it working? (other than setting api 7 or 8 in the entry tp file).

Currently I have gotten the updated broadcast message, and the new tooltip on settings working.
But actions get broken with the new entry.tp setup (lines insted of format etc.), and stateListUpdate and triggerEvent

@KillerBOSS2019
Copy link
Owner Author

Hi, @oddbear ,
I was working to support v7 api which was a lot of work due to entry validation with deprecated api for v7 and backwards compatibility. That being said, it's mostly stable. But I haven't done enough testing. And I don't really want to switch new entry format. So there's that....

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.

2 participants