-
Notifications
You must be signed in to change notification settings - Fork 4
Conversation
4d876c7
to
0c88d1a
Compare
0c88d1a
to
536fe1a
Compare
536fe1a
to
23d56b9
Compare
23d56b9
to
c8f7232
Compare
259b15e
to
db4387d
Compare
5649290
db4387d
to
5649290
Compare
@@ -107,7 +107,7 @@ export default class requestController { | |||
passportName, | |||
role | |||
}], | |||
status: 'Open' | |||
status: 'open', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
which case did we decide to go with?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lower case is much better so that's what we are going with
src/controllers/tripsController.js
Outdated
@@ -49,7 +49,7 @@ export default class requestController { | |||
passportName, | |||
role | |||
}], | |||
status: 'Open' | |||
status: 'Open', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then this should change too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let me change the code so that the data always inter the database as small letter even tho it wasn't entered as small letters
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I solved it, now when someone input data like a city, it will directly be converted to small letters
- add the reconfirm controller and route - test the feature - add the managerId and confirm attribute to the request table [Maintains #170947570]
5649290
to
4ecfacd
Compare
What does this PR do?
add the re-confirm feature where a manager is able to confirm a decision (accepted or rejected) on a request
Description of Task to be completed?
How should this be manually tested?
Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?
#170947570
Screenshots (if appropriate)
Questions: