Tune inline edit text input

keboola / indigo-ui

Indigo UI library, its CSS and React components. Also Styleguide app showing usages of this library.

https://indigo.keboola.com

MIT License

5 stars 0 forks source link

Tune inline edit text input #469

Closed cjayyy closed 3 years ago

cjayyy commented 3 years ago

Changes:

Add default true value to isValid prop
Remove events and prevent default in onEditStart and onEditCancel handlers

ref.: https://github.com/keboola/kbc-ui/pull/6364#discussion_r713616061

natocTo commented 3 years ago

@cjayyy toto klidně merge a může to být jako 17.1.0 asi možná (když se mění ty parametry props někdy).

cjayyy commented 3 years ago

Sakra, nevím jak se tady nasazuje. Pushnul jsem verzi, dál tohle mergnu, ale potom nevím co?

natocTo commented 3 years ago

Teď jsem to taky dlouho nedělal ale neupravoval bych ten package.json.

Mergnul bych jen tu změnu pak šel přes github -> releases -> draf new release -> create new v17.1.0, dopsat popis co je upraveno a release.

cjayyy commented 3 years ago

Teď jsem to taky dlouho nedělal ale neupravoval bych ten package.json.

Mergnul bych jen tu změnu pak šel přes github -> releases -> draf new release -> create new v17.1.0, dopsat popis co je upraveno a release.

Ten package.json upravuje yarn version kterej jsem dal podle návodu.

cjayyy commented 3 years ago

Udělám to jak píšeš.

natocTo commented 3 years ago

Ono to je asi podobný, že pak budeš mít na výběr tu verzi v githubu a nebudeš muset vytvářet.

cjayyy commented 3 years ago

Ještě jednou approve prosím. (Kvůli tomu že jsem smazal poslední commit)

cjayyy commented 3 years ago

NPM publish končí tímto errorem:

npm ERR! 403 403 Forbidden - PUT https://registry.npmjs.org/@keboola%2findigo-ui - You cannot publish over the previously published versions: 17.0.0.
npm ERR! 403 In most cases, you or one of your dependencies are requesting
npm ERR! 403 a package version that is forbidden by your security policy.

Že by se tam nějak nespároval ten release?