search

nswalters / AppTrakz-Client

React Client for AppTrakz application for documenting the job application process.
0 stars 0 forks source link

Update Application #23

Closed nswalters closed 3 years ago

nswalters commented 3 years ago

As a user, I would like to be able to edit an application in case of an error or a change is required such as a status update.

Given a user is viewing the application list OR a specific application When they select the option to edit an application Then they should be directed to a form and given the ability to change the application information

Given the user is finished updating the application information When they click the Save button Then the application information should be updated in the database And the user should be redirected to the application details view

Given the user has decided to not edit the application information When they click the Cancel button Then they should be redirected to the application details view

Editable fields should include:

nswalters commented 3 years ago

Every time an application status is updated, we should create a brand new database entry. This way, we can track the changes over time. If we just update the record in place, then we lose the history. We will need the history to generate any type of graphing over time.

To do this, you would need to grab all of the application statuses for a specific application and then grab the one with the most recent update time.

OR we can add another field on to the application_status records called "is_current" and if we need to create a new record, we create that new record and set the old record 'is_current' as false. Then all we need to do is grab the status for an application where is_current = true. I feel like this way is cleaner.

nswalters commented 3 years ago

OR we can add another field on to the application_status records called "is_current" and if we need to create a new record, we create that new record and set the old record 'is_current' as false. Then all we need to do is grab the status for an application where is_current = true. I feel like this way is cleaner.

This was the route taken and is handled by the backend already. See https://github.com/nswalters/AppTrakz-API/pull/16 for details on what actually needs to be submitted to create an updated application record.