Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add middleware for setting lastEdited and lastEditedBy #264

Open
mattwalo32 opened this issue Jul 16, 2021 · 1 comment
Open

Add middleware for setting lastEdited and lastEditedBy #264

mattwalo32 opened this issue Jul 16, 2021 · 1 comment
Labels
P2 Low Priority refactoring

Comments

@mattwalo32
Copy link
Collaborator

This may not be possible, but it's worth looking into

@mattwalo32 mattwalo32 added P2 Low Priority refactoring labels Dec 14, 2021
@ashayp22
Copy link
Collaborator

ashayp22 commented Jan 6, 2022

I was going to add this to #550, but I was unable to think of a way to set up a middleware. This is because lastEdited and lastEditedBy is set after getting patientData and patientStepData in the body of the function for a specific request. For #550, I initially set up a middleware that sets lastEdited for Note/MultiLine fields that require it (and not step and patient), but I then moved this to the frontend since it allowed lastEdited to only update after the user changes the field. It would be overcomplicated to update lastEdited on the backend in this case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 Low Priority refactoring
Projects
None yet
Development

No branches or pull requests

2 participants