-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: implement register consent flow #79
Conversation
65c4c3f
to
0296f7c
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.
Looks good, minor nitpicks.
I feel like all this time working on getting the state machines right is paying off.
const payload: fspiopAPI.Schemas.ParticipantsTypeIDSubIDPostRequest = { | ||
fspId: this.config.authServiceParticipantFSPId | ||
} | ||
const res = await axios.post(alsParticipantURI, payload, axiosConfig) |
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.
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.
We don't have type definitions for BaseRequests
and instead of defining all the types this was easier until I can commit a postParticipantsTypeId
function to sdk-standard-components
.
import ParticipantsTypeIDHandler from '~/server/handlers/participants/{Type}/{ID}' | ||
import { StateResponseToolkit } from '../../../../../../src/server/plugins/state'; |
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.
Any reason to use these instead of the ~
?
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.
auto import in vscode imports them like this and I sometimes miss it.
been meaning to find a way to fix that.
Going to merge this for now o/ |
No description provided.