search

massdriver-cloud / massdriver-cli

Deprecated. See https://github.com/massdriver-cloud/mass
https://massdriver.cloud
Apache License 2.0
3 stars 2 forks source link

Mass app new require description #167

Closed chrisghill closed 1 year ago

chrisghill commented 1 year ago

This is fixing some issues I've scene customers run into when setting up apps.

Apps are likely stored in the same directory as the application code itself, so a massdriver subdirectory (app/massdriver/massdriver.yaml) makes more sense than using the application name as the subdirectory (app/app/massdriver.yaml).

If the description field is empty it gets rejected by the API, so we should prevent empty descriptions when creating a new app/bundle.

WillBeebe commented 1 year ago

Thank you! I've been kicking these headaches down the road.

WillBeebe commented 1 year ago

Looks pretty shippable to me, good work.

@coryodaniel annoying ping to check that this is in V2 or slated.