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

Vampirism web api #1220

Merged
merged 5 commits into from
Aug 9, 2023
Merged

Vampirism web api #1220

merged 5 commits into from
Aug 9, 2023

Conversation

Cheaterpaul
Copy link
Member

@Cheaterpaul Cheaterpaul commented Jun 25, 2023

  • swap contributor api
  • allows to enable/disable features remotely in the api
  • sends basic telemetry data, if it is not disabled in minecraft and vampirism
    • mod version
    • minecraft version
    • installed mods
    • server/client
  • automatic update of config values and contributors if pushed to the default branch

@Cheaterpaul Cheaterpaul marked this pull request as ready for review June 25, 2023 16:32
Copy link
Member

@maxanier maxanier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice improvement

- settings and supporter only have optional return types
- rename non api methods
- remove api version as extra attribute
- rename config_values.json to default_remote_config.json
@Cheaterpaul Cheaterpaul merged commit cb8fab0 into dev Aug 9, 2023
1 check passed
@Cheaterpaul Cheaterpaul deleted the feature/settings_api branch September 19, 2023 13:36
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.

2 participants