Skip to content

Make other API available in the playground#1246

Merged
richvdh merged 4 commits into
masterfrom
playground-all-apis
Feb 16, 2022
Merged

Make other API available in the playground#1246
richvdh merged 4 commits into
masterfrom
playground-all-apis

Conversation

@afranke
Copy link
Copy Markdown
Contributor

@afranke afranke commented Feb 2, 2022

This adds a combo that allows selection of API amongst the ones we generate.

Depends on matrix-org/matrix-spec-proposals#3684

Fixes matrix-org/matrix-spec-proposals#1458

Preview: https://pr1246--matrix-org-previews.netlify.app

Alexandre Franke added 3 commits February 2, 2022 14:06
Signed-off-by: Alexandre Franke <alexandre.franke@matrix.org>
Signed-off-by: Alexandre Franke <alexandre.franke@matrix.org>
Signed-off-by: Alexandre Franke <alexandre.franke@matrix.org>
@afranke afranke requested a review from a team February 2, 2022 16:34
Signed-off-by: Alexandre Franke <alexandre.franke@matrix.org>
Copy link
Copy Markdown
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

seems generally sensible to me

@richvdh richvdh merged commit e15c197 into master Feb 16, 2022
@richvdh richvdh deleted the playground-all-apis branch February 16, 2022 18:01
@afranke
Copy link
Copy Markdown
Contributor Author

afranke commented Feb 17, 2022

Something’s amiss.

Looks like it didn’t deploy? 17 hours after it’s been merged, doesn’t seem right…

@richvdh
Copy link
Copy Markdown
Member

richvdh commented Feb 17, 2022

hrm, thanks @afranke; will take a look

@richvdh
Copy link
Copy Markdown
Member

richvdh commented Feb 17, 2022

so it turns out the webhook listener was ooming. Hopefully that is now improved with matrix-org/gha-webhook-listener#8.

@afranke
Copy link
Copy Markdown
Contributor Author

afranke commented Feb 17, 2022

Thanks for looking into it and for fixing it so quickly. Had a look at the hook changes, they look good.

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.

Swagger viewer for all the APIs

2 participants