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

Initial MSC1929 Implementation #150

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

Conversation

MTRNord
Copy link
Collaborator

@MTRNord MTRNord commented Oct 19, 2023

Adds a command to query the MSC1929 JSON and display it. It also adds tests for this.


Some example output:

All infos available:

Support info for (http://localhost:8081):
Support Page: http://localhost

Only users available:

Support info for (http://localhost:8081):

Only Support page available:

Support info for (http://localhost:8081):
Support Page: http://localhost

@MTRNord

This comment was marked as outdated.

@MTRNord MTRNord mentioned this pull request Oct 20, 2023
9 tasks
@MTRNord MTRNord marked this pull request as ready for review October 24, 2023 12:46
@MTRNord MTRNord requested a review from Gnuxie October 24, 2023 12:49
@MTRNord MTRNord removed their assignment Oct 24, 2023
src/commands/QueryAdminDetails.tsx Outdated Show resolved Hide resolved
src/commands/QueryAdminDetails.tsx Outdated Show resolved Hide resolved
src/commands/QueryAdminDetails.tsx Outdated Show resolved Hide resolved

try {
const resp: SupportJson = await new Promise((resolve, reject) => {
getRequestFn()(`${domain}/.well-known/matrix/support`, (error: any, response: any, resBody: unknown) => {
Copy link
Member

Choose a reason for hiding this comment

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

image

what did i do wrong here? do they just not support the MSC?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ah Hm ok yeah it prints the error but seems like in this case it just printed the whole page oO. Yeah not supported and it went wrong.

src/commands/QueryAdminDetails.tsx Outdated Show resolved Hide resolved
src/commands/QueryAdminDetails.tsx Outdated Show resolved Hide resolved
test/nginx.conf Show resolved Hide resolved
mx-tester.yml Show resolved Hide resolved
@Half-Shot
Copy link
Contributor

N.B. We updated the MSC with a few name changes here and there, you might want to update to reflect changes.

@MTRNord
Copy link
Collaborator Author

MTRNord commented Nov 21, 2023

N.B. We updated the MSC with a few name changes here and there, you might want to update to reflect changes.

I will take a look. I think I already applied the comments before the MSC did :) Thanks for the notice however!

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.

3 participants