Support for new native types manifest

keboola / php-component

General library for PHP applications running in Keboola Connection environment

MIT License

0 stars 1 forks source link

Support for new native types manifest #82

Closed AdamVyborny closed 1 month ago

AdamVyborny commented 2 months ago

JIRA: https://keboola.atlassian.net/browse/PST-1376

Nakonec jsem to tedy překopal na to, že ten objekt je pouze v novém formátu, ale zachoval jsem původní settery (které hlásí deprecation notice) abychom měli snadnější update těch komponent. Načíst manifest ze souboru také zvládám z obou formátů a vygenerovat ho taky do obou. Asi narazíme ještě na nějaký edgecase, protože do teď byl ten formát manifestů totálně volný, ale jinak by to takhle mohlo fungovat...

AdamVyborny commented 2 months ago

@odinuv Máme už někde vzorový .manifest abych ho přidal do těch funkcionálních testů na read a write? V RFC je jen ten minimal nebo ta verze z rozepsanými změnami.

odinuv commented 1 month ago

Jinak finální podoba manifestu by měla bejt tady https://github.com/keboola/platform-libraries/blob/main/libs/output-mapping/tests/Configuration/Table/BaseConfigurationTest.php a tady https://github.com/keboola/platform-libraries/blob/main/libs/output-mapping/tests/Configuration/Table/ManifestTest.php připadně se tam může ještě doplnit nějaká další

ErikZigo commented 1 month ago

Na revíew dělá OJ a po dohodě se odebírám. On do toho má větší vhled. Kdyby byla nějaká nejasnost, tak mě sem zas přidejte.