-
Notifications
You must be signed in to change notification settings - Fork 42
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
Admin should be able to edit mentor details #168
Comments
@Disura-Randunu Would you like to work on this? |
@anjula-sack Sure. Should I prioritize this issue or #155 |
You can prioritize this |
@anjula-sack img-url is a property of profile. Shall I make it possible to update other profile data within the mentor update as well? |
@anjula-sack Also, since there's an image, it will be a form-data request. The issue is with type conversions specially with key-value pairs in the "application" property. those values will always be strings since we're passing them through a form data like below.
|
Yeah let's go with first option since it's more flexible and we probably will refactor how we are handling the applications. @Disura-Randunu |
@anjula-sack what about the profile data other than image? Should they be updatable within mentor update? |
yes |
@anjula-sack added PR |
Description:
Tasks:
Acceptance Criteria:
The text was updated successfully, but these errors were encountered: