Skip to content
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: update translation key create/update with custom metadata. Add CM schema in translation key response #480

Conversation

anthonyshchang
Copy link
Contributor

@anthonyshchang anthonyshchang commented Dec 7, 2023

Description

update translation key create/update with custom metadata. Also. add CM in translation key schema response

response CM in translation key
Screenshot 2023-12-12 at 14 20 33

Update/Create CM on translation key

Screen.Recording.2023-12-07.at.10.34.10.mov

Ticket

https://phrase.atlassian.net/browse/TSS-2504

@anthonyshchang anthonyshchang marked this pull request as ready for review December 7, 2023 09:36
Copy link
Collaborator

@jablan jablan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

docs for update should be clarified IMO

paths/keys/update.yaml Outdated Show resolved Hide resolved
@forelabs forelabs added the DONT MERGE indicates that the PR should not (yet) be merged label Dec 12, 2023
@ildarsafin
Copy link
Contributor

@forelabs we can merge this pr even today if possible since its going to be merged to TSS-1890-rest-api-index branch

@ildarsafin ildarsafin changed the title feat: update translation key create/update with custom metadata feat: update translation key create/update with custom metadata. Add CM schema in translation key Dec 12, 2023
@ildarsafin ildarsafin changed the title feat: update translation key create/update with custom metadata. Add CM schema in translation key feat: update translation key create/update with custom metadata. Add CM schema in translation key response Dec 12, 2023
@forelabs
Copy link
Member

Will have a look in a second

@BlumMichael BlumMichael removed their request for review December 12, 2023 14:15
@ildarsafin ildarsafin removed the DONT MERGE indicates that the PR should not (yet) be merged label Dec 12, 2023
@ildarsafin ildarsafin merged commit c42fbea into TSS-1890-rest-api-index Dec 12, 2023
13 checks passed
@ildarsafin ildarsafin deleted the TSS-2504-update-api-docs-for-translation-key-custom-metadata branch December 12, 2023 16:37
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 <matias.alvarez@phrase.com>

* 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 <knock@ildarsafin.tech>

* 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 <matias.alvarez@phrase.com>
Co-authored-by: Ildar Safin <knock@ildarsafin.tech>
Co-authored-by: Mladen Jablanovic <mladen.jablanovic@phrase.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants