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

Matrix chat #410

Open
1 task done
suprohub opened this issue Dec 25, 2024 · 4 comments
Open
1 task done

Matrix chat #410

suprohub opened this issue Dec 25, 2024 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@suprohub
Copy link
Contributor

I've searched existing issues and couldn't find a duplicate.

  • I confirm this is not a duplicate.

Is your feature request related to a problem? Please describe.

Matrix chat can be good alternative to current apps like discord

Describe the solution you'd like

Create matrix chat

@suprohub suprohub added the enhancement New feature or request label Dec 25, 2024
@DataM0del
Copy link
Contributor

DataM0del commented Dec 25, 2024

Hello, we tried to solve the issue.

This is what we did:

Added a new field 'matrix' to the ServerLinksConfig struct to support Matrix chat links. Updated the Default implementation to include an empty string for the matrix field.

You can review changes in this commit: jacks-sam1010@a3e5c45.

Caution

Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

Latta AI seeks to solve problems in open source projects as part of its mission to support developers around the world. Learn more about our mission at https://latta.ai/ourmission . If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.

gullible AI moment.
You aren't supposed to add any fields to ServerLinksConfig, since it should only contain fields that the Notchian (vanilla) Minecraft client supports, for other links, you would add them to the custom hashmap.
Anyways, they meant to create a matrix chat for Pumpkin.

@Commandcracker
Copy link
Contributor

@jacks-sam1010

This is not how it works jacks-sam1010@a3e5c45.
See https://minecraft.wiki/w/Minecraft_Wiki:Projects/wiki.vg_merge/Protocol#Server_Links_(configuration).
Just use custom if you want to add anything, no changes required.

@suprohub
Your question is not that precise, did you mean to create an official Matrix group for pumpkin ?
I don't think we will do that because moderating matrix is harder than discord and I don't think we want to have extra moderation effort.

@suprohub
Copy link
Contributor Author

pofig

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants
@Commandcracker @Snowiiii @suprohub @DataM0del and others