feat: add delete CM property doc #477
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
anthonyshchang
requested review from
forelabs,
malvarezphrase and
BlumMichael
anthonyshchang
changed the base branch from
master
to
TSS-1890-rest-api-index
malvarezphrase
approved these changes
Dec 4, 2023
anthonyshchang
added
the
DONT MERGE
anthonyshchang
force-pushed
the
TSS-2530-create-delete-doc
branch
from
961f2b4 to
2263b9d
Compare
malvarezphrase
approved these changes
Dec 6, 2023
| @@ -50,7 +50,6 @@ | |||
| "/accounts/{account_id}/custom_metadata/labels": | |||
| get: | |||
| "$ref": "./paths/custom_metadata_properties/index.yaml" | |||
| <<<<<<< HEAD | |||
There was a problem hiding this comment.
How did this end up there? :D
ildarsafin
added a commit
that referenced
this pull request
Dec 13, 2023
* register Custom Metadata Labels to main yml * add custom metadata schema * add custom metadata index path * add enum for data_type * specify 2.9 cli version * generalize custom metadata tag * correct path for index route * update path * update compiled json * align Custom Metadata tag usage * add project ids param to labels index * add sort and order to metadata label index * update url params example * make summary and description more clear and concise * add endpoint availability notice * remove account ref from schema * add projects to custom metadata label schema * update label -> property * rename CM paths folder * fix parameter definitions * feat(api): Add show API endpoint for custom_metadata_labels (#359) * add paths for custom metadata show * make summary and description more clear and concise * fix path with namespace * change from label to property for consistency * remove unused branch param --------- Co-authored-by: Matias Alvarez <[email protected]> * feat: add CM api update (#475) * add name and description * add create api doc for CM properties (#476) * feat: add delete CM property doc (#477) * add delete api doc for CM properties * rename data type to custom metadata data type * nest create under collection * update routes to use properties instead of labels * fix project_id example * update path CM route naming * feat: update translation key create/update with custom metadata. Add CM schema in translation key response (#480) * add CM to create/update keys * Add custom metadata object on CM get key * Add more description for update --------- Co-authored-by: Ildar Safin <[email protected]> * Make create CM example to be working with CURL * Make the data_type field to be not required * extract custom metadata data type schema --------- Co-authored-by: Matias Alvarez <[email protected]> Co-authored-by: Ildar Safin <[email protected]> Co-authored-by: Mladen Jablanovic <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Screen.Recording.2023-12-02.at.23.30.24.mov
DONT MERGE
DONT MERGE UNTIL FEATURE TOGGLE IS REMOVED!
Ticket
https://phrase.atlassian.net/browse/TSS-2530