Skip to content
This repository was archived by the owner on Mar 12, 2025. It is now read-only.

PATCH skills/{skillId} endpoint does not allow to patch the metadata fields #8

Closed
SathyaJayabal opened this issue Aug 11, 2021 · 4 comments
Closed

PATCH skills/{skillId} endpoint does not allow to patch the metadata fields #8

SathyaJayabal opened this issue Aug 11, 2021 · 4 comments
Assignees
@Sande3p
Labels
p0 Highest priority issues QA Pass in Dev QA Pass in Prod
Milestone
v0.1.0

Comments

@SathyaJayabal
Copy link
Collaborator

PATCH skills/{skillId} endpoint does not allow to patch the metadata fields

we get the following error when trying to patch data with meta data fields
Screenshot 2021-08-11 at 2 51 39 PM
@vikasrohit
Copy link

Patch should be allowed for metadata fields of both entities.

@vikasrohit vikasrohit added the p0 Highest priority issues label Aug 16, 2021
@vikasrohit
Copy link

@Sande3p is this one fixed?

@vikasrohit vikasrohit added the Ready for Dev QA Ready for QA in dev environment label Aug 18, 2021
@SathyaJayabal
Copy link
Collaborator Author

Verified on dev, able to patch with metadata fields
Screenshot 2021-08-18 at 5 17 37 PM

@SathyaJayabal SathyaJayabal added QA Pass in Dev and removed Ready for Dev QA Ready for QA in dev environment labels Aug 18, 2021
Sande3p added a commit that referenced this issue Aug 19, 2021
@Sande3p
fix for #8 #13
52c35c3
@SathyaJayabal SathyaJayabal added this to the v0.1.0 milestone Aug 19, 2021
@vikasrohit vikasrohit added the Ready for Prod QA Ready for QA in prod environment label Aug 19, 2021
@SathyaJayabal SathyaJayabal added QA Pass in Prod and removed Ready for Prod QA Ready for QA in prod environment labels Aug 19, 2021
@SathyaJayabal
Copy link
Collaborator Author

verified on prod
Screenshot 2021-08-19 at 6 31 46 PM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Sande3p Sande3p

Labels
p0 Highest priority issues QA Pass in Dev QA Pass in Prod
Projects
None yet
Milestone
v0.1.0
Development

No branches or pull requests
3 participants
@vikasrohit @Sande3p @SathyaJayabal