-
Notifications
You must be signed in to change notification settings - Fork 5
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
Improve existing logical operation #51
Improve existing logical operation #51
Conversation
b6d7aa8
to
bd94c4e
Compare
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.
Hi @NiyoEric , I have left some comments, please view them
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.
Well done!
bd94c4e
to
d963acd
Compare
29361c9
to
a559d8b
Compare
- refactor code - add style - [Finishes #171403437]
a559d8b
to
a08905c
Compare
d0f91c3
to
2f405ab
Compare
2f405ab
to
df81867
Compare
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.
Well done @NiyoEric
However, I've left some feedback. We can work together to get it done!
- improve current condition checking - update env modify multicity trip request - remove string comparison for trip request - check the status of the trip - edit test` - [Finishes #171403437]
7a6d9a2
to
5c6d6a0
Compare
What does this PR do?
The PR helps to refactor the existing work.
Description of Task to be completed?
string comparisonHow should this be manually tested?
git clone https://github.com/andela/team-odd-bn-frontend.git
cd team-odd-bn-backend
npm install
Any background context you want to provide?
The current work works well. However, we need to improve.
What are the relevant pivotal tracker stories?
#171403437
Screenshots (if appropriate)
Questions: