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

Port to deploy/hammer #748

Closed
wants to merge 6 commits into from
Closed

Port to deploy/hammer #748

wants to merge 6 commits into from

Conversation

aaronchongth
Copy link
Member

What's new

Ports

Self-checks

  • I have prototyped this new feature (if necessary) on Figma
  • I'm familiar with and follow this Typescript guideline
  • I added unit-tests for new components
  • I tried testing edge cases
  • I tested the behavior of the components that interact with the backend, with an e2e test

aaronchongth and others added 6 commits August 21, 2023 21:22
* fix login page route

Signed-off-by: Teo Koon Peng <[email protected]>

* also fix default route

Signed-off-by: Teo Koon Peng <[email protected]>

* fix redirect

Signed-off-by: Teo Koon Peng <[email protected]>

---------

Signed-off-by: Teo Koon Peng <[email protected]>
* Rename place to waypoint

Signed-off-by: Aaron Chong <[email protected]>

* Reduce delivery form requirements to just pickup and dropoff waypoint names

Signed-off-by: Aaron Chong <[email protected]>

* Adding WaypointCollection, which filters each waypoint type, display waypoint options only for that type

Signed-off-by: Aaron Chong <[email protected]>

* Disable menu item if task is unavailable

Signed-off-by: Aaron Chong <[email protected]>

* Fix error of passing a value to element by int instead of string

Signed-off-by: Aaron Chong <[email protected]>

* Using patrol instead of loop

Signed-off-by: Aaron Chong <[email protected]>

* Fix documentation and create-task story

Signed-off-by: Aaron Chong <[email protected]>

* Revert to Place, as Waypoint implies routes

Signed-off-by: Aaron Chong <[email protected]>

* Revert changes to variable names

Signed-off-by: Aaron Chong <[email protected]>

* Refactor if case

Signed-off-by: Aaron Chong <[email protected]>

---------

Signed-off-by: Aaron Chong <[email protected]>
* Fixing missing handler delivery task description

Signed-off-by: Aaron Chong <[email protected]>

* Fix lint

Signed-off-by: Aaron Chong <[email protected]>

* Disabled delivery when either pickup or dropoff points are empty

Signed-off-by: Aaron Chong <[email protected]>

---------

Signed-off-by: Aaron Chong <[email protected]>
* Add routes to delete only one job

Signed-off-by: angatupyry <[email protected]>

* Add except date to cancel a job

Signed-off-by: angatupyry <[email protected]>

* Testing clear only one job

Signed-off-by: angatupyry <[email protected]>

* Remove unused endpoint

Signed-off-by: angatupyry <[email protected]>

* Delete all jobs and create again when except date is included this week

Signed-off-by: angatupyry <[email protected]>

* Show event or not depending of the except date

Signed-off-by: angatupyry <[email protected]>

* Fix comment to explain the code better

Signed-off-by: angatupyry <[email protected]>

* Add except date to the Schedule task and remove from ScheduleTaskschedule

Signed-off-by: angatupyry <[email protected]>

* Create a endpoint to add a except date to the array and check whether or not the curretn date is include in the array of except_dates

Signed-off-by: angatupyry <[email protected]>

* Adding control in order to show events

Signed-off-by: angatupyry <[email protected]>

* Add confirmation popup to delete one event or all events

Signed-off-by: angatupyry <[email protected]>

* Sort alphabetical order

Signed-off-by: angatupyry <[email protected]>

* Remove console.log

Signed-off-by: angatupyry <[email protected]>

* Using throw error to show message in console

Signed-off-by: angatupyry <[email protected]>

* Change the endpoint and endpoint parameters to make them more accurate and understandable.

Signed-off-by: angatupyry <[email protected]>

---------

Signed-off-by: angatupyry <[email protected]>
@aaronchongth
Copy link
Member Author

closing in favor of #749

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