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

Lines refactoring, reservations refactoring and some bug fixes #52

Merged
merged 12 commits into from
Oct 12, 2019

Conversation

simonasaitta
Copy link
Collaborator

@simonasaitta simonasaitta commented Oct 10, 2019

Line refactoring
Now the lines structure follows the specifics and has a list of stops containing each time for the outward and backward runs. (Issue #43)

Reservations refactoring
Reservations now follow the specifics and allow to delete a reservation and reserve the default stop/line if available with one click. If there is no available default line it will allow the select one. Long press on a reservation allows changes to it.
Reservations are now allowed only before the run. (Issue #47, #32, #38 )

Bug fixes
Fixed bug showing previous profile when logging back with a different one.
Toolbar now shows user picture.

New reservations screenshots
image
image
image
image
image
image

Lines now have a list of stops with a list of times for outward and backward routes.
# Conflicts:
#	frontend/src/app/components/profileInfo/change-password/change-password.component.html
#	frontend/src/app/components/profileInfo/profile/profile.component.html
# Conflicts:
#	frontend/src/app/services/reservations/reservations.service.ts
…servations. Fixed bug in profile when removing selected default line/stop.
# Conflicts:
#	frontend/src/app/components/attendance/attendance.component.ts
#	pom.xml
@simonasaitta simonasaitta requested a review from daevem October 10, 2019 16:55
@daevem daevem merged commit 330501c into suidpit:master Oct 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants