Cockpit-HQ / Cockpit

Cockpit Core - Content Platform
https://getcockpit.com
Other
388 stars 47 forks source link

[fix] select field renderer #166

Closed falk-m closed 11 months ago

falk-m commented 11 months ago

if options in select list are defines as object with labels, then display the labels

165

aheinze commented 11 months ago

Thank you 🙏

falk-m commented 11 months ago

Hi @aheinze , ich sehe du sitzt in Hamburg. Arbeitest du beruflich an Cockpit?

Ich wollte schon immermal ein Projekt mit Cockpit bauen und jetzt ist es endlich so weit. Ich finde den Mongolight Ansatz mit sqLight sehr genial.

Ich verstehe nicht ganz, was eure code quality hier dran zu meckern hat:

const selectedOptions = options.filter((o) => selectedValues.indexOf((o.value ?? o).toString().trim()) !== -1);
falk-m commented 11 months ago

Okay, ich sehe dass es eh schon gemerged ist:-) 👍

aheinze commented 11 months ago

Ja, ich mach gleich noch einen Cleanup. Super, dass dir Cockpit soweit gefällt. Cockpit habe ich für eigene Projekte/Kunden entwickelt. Andere scheinen es jedenfalls auch zu mögen :-)