"DRY" the forms - Githubissues

djstevenson / songs-to-the-siren

A blog about songs

MIT License

0 stars 0 forks source link

"DRY" the forms #205

Closed djstevenson closed 4 years ago

djstevenson commented 4 years ago

A load of forms have a near-identical "Create" and "Delete" form definition. Merge these into one. Probably by making the object optional, and using a predicate to determine if we're doing a create or an update.

djstevenson commented 4 years ago

List of form pairs:

Form::Content::Create/Edit
Form::Song::Create/Edit
Form::Link::Create/Edit
Form::Country::Create/Edit

djstevenson commented 4 years ago

There are only four such form pairs and I need to get this into production, so this is worth looking at but not as a high priority.