Skip to content
This repository has been archived by the owner on Apr 11, 2021. It is now read-only.

Minecraft API #40

Open
5 tasks
stachu540 opened this issue Jul 10, 2019 · 3 comments
Open
5 tasks

Minecraft API #40

stachu540 opened this issue Jul 10, 2019 · 3 comments
Assignees
Labels
docs: required Documenting some part of Pull Request's or reproducing issue feature: chat New feature or request feature: core New feature or request feature: helix New feature or request feature: kraken New feature or request feature: pub-sub New feature or request
Milestone

Comments

@stachu540
Copy link
Collaborator

Description

For Minecraft programmers, will be prepared some solutions with specific module. Discuss over here is open. Of course feel to discuss more details on our Discord. This will help improve our code to solving solutions.

Steps to Reproduce

  • Research a most useful API Add-on for Minecraft
  • Search Kotlin solutions
  • Code to the API
  • Utilize test specification
  • Deploy to server for Real Test

Expected behavior:
Listening required data.
Actual behavior:
Code requires app data which is not required to create listener.
Stacktrace (if applicable):
only if using .doOnError {}
Version affected:
1.x

@stachu540 stachu540 added this to the v1.x milestone Jul 10, 2019
@stachu540 stachu540 self-assigned this Jul 10, 2019
@stachu540 stachu540 added this to To do in v1.x via automation Jul 10, 2019
@stachu540 stachu540 added docs: required Documenting some part of Pull Request's or reproducing issue feature: chat New feature or request feature: core New feature or request feature: helix New feature or request feature: kraken New feature or request feature: pub-sub New feature or request help wanted Extra attention is needed labels Jul 10, 2019
@ShayBox
Copy link

ShayBox commented Jul 10, 2019

Current workaround: Provide empty strings for GlitchClient#setClientId and GlitchClient#setClientSecret

@stachu540
Copy link
Collaborator Author

After 0.7.0 which means whole code will be changed, i can able to provisoring GlitchClient as Void with CredentialManager support

@stachu540 stachu540 removed the help wanted Extra attention is needed label Jul 17, 2019
@stachu540
Copy link
Collaborator Author

Referring to this response. Client ID is still required for calling API request's. chat module needs them to gathering bot information to build Rate-Limiting for this module. PubSub I guess will not need them cause he needs only accessToken and required Scope to handling required Topic.

Another solution is refreshing token. When your accessToken calls Unathorized they will forcing regenerate your Credential. Still will saving to CredentialStorage replace to authorized user via id.

Why by id? Twitch last time add changing name solutions, that's why.

Why i need them (clientId and clientSecret)? If your token expire, script will try refreshing them following by twitch dev documentation

Conclusion: PubSub don't needs a API calls. This is a Optional solutions, soo, avoiding expired token is not a option. In case. Token expire, refresh it. To refreshing them clientId and clientSecret needs soo bad.

@stachu540 stachu540 removed this from To do in v1.x Jul 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docs: required Documenting some part of Pull Request's or reproducing issue feature: chat New feature or request feature: core New feature or request feature: helix New feature or request feature: kraken New feature or request feature: pub-sub New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants