programsam / binkjs

Music archival site BINK, rewritten as a Node.js express app
0 stars 1 forks source link

Edit a jam #90

Closed programsam closed 3 years ago

programsam commented 6 years ago

This is an epic if we had them.

There is an issue open for creating a jam (#56), which I think you might argue is done, but you have to make sure the flow from creation to editing is smooth.

https://cdnjs.com/libraries/bootstrap-select

Bootstrap select is a searchable select text box that should make it easier to add musicians, instruments, staff, set locations, set bands and so on. Critical to the edit functionality.

As far as architecture I think it makes sense to have a create call that inserts a new row to the table, and then the rest of the admin experience is achieved by a series of automatically applied Ajax calls that update that individual thing.

So for example, in edit mode, you change the title, this results in an Ajax PUT that says update jam 163 with this title, date, description and private status. That’s all that goes in the call.

Then if you want to add a musician, that’s a separate Ajax POST or PUT that contains the musician id, jam id, and probably instrument. Needs to be possible to easily add new entities in line from the edit screen.

A Save button exits from edit mode but probably doesn’t make any calls that edit the database.

programsam commented 5 years ago

Getting started with this:

https://github.com/programsam/binkjs/commit/a07c24e5b7e6dd0d6ec96b0ad99d233dc0359774

programsam commented 3 years ago

Jams are now editable.