-
Notifications
You must be signed in to change notification settings - Fork 149
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
Refactor provisioning API #1655
Conversation
f2e0c83
to
ea61853
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing here is really jumping out at me as obviously bad so far 👍 . Suggest continuing to crack on with the widget development (in a child PR?) and circling back to the tests here afterwards.
b619e23
to
c88956e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's gonna look like I didn't even read it – but it does look good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy happy from my PoV
(Much of these changes derived from #1508)
Makes significant updates to the Provisioning API implementation, namely refactoring to use the
ProvisioningApi
class frommatrix-appservice-bridge
.Also implements request validation and consistent error handling across the API.