chore: remove endpoint from OpenAPI spec#203
chore: remove endpoint from OpenAPI spec#203Verifieddanny wants to merge 1 commit intoRocketChat:mainfrom
Conversation
|
i think we need to change the branch from main to the branch that im working on #150 |
|
@ggazzo 👍 |
|
The branch you're working on, I believe is from your forked repo And being compared to main branch on the OpenAPI repo (do you suggest I clone your forked repo and push from there instead?) |
Personally, I don't mind that approach, but I suspect the maintainers would prefer us to create a dedicated branch within the main repository. That way, we can all merge our PRs into a single shared branch before the final merge into |
Ohh Alright, I think that approach is good, I will wait for the supposed branch to be created |
|
Hey @ahmed-n-abdeltwab , how do we move forward from here ? Are you still creating a dedicated branch |
for now i recommend change the name instead of remove rooms.leave endpoint to remove endpoint. and until we get further clarification on their plans. We’re following the current instructions, so just keep updating this PR as you finish each API |
|
Alright, noted |
Remove endpoints from OpenAPI spec
The
rooms.leaveendpoint has been migrated to the new OpenAPI-compliantpattern with AJV validation in the main Rocket.Chat repository.
Related PR: RocketChat/Rocket.Chat#38957 (merged)
Tracking PR: #150