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.

Sign up for GitHub

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

pipes - Tamira - BackTREK #23

Open
wants to merge 46 commits into
base: master
Choose a base branch
from
Open

pipes - Tamira - BackTREK #23

wants to merge 46 commits into from

Commits on Nov 28, 2017

  1. made Trip model and TripList collection

    @tvojnar
    tvojnar committed Nov 28, 2017
    Configuration menu
    Copy the full SHA
    93337ca View commit details
    Browse the repository at this point in the history

  2. all trip are displayed on the page automatically

    @tvojnar
    tvojnar committed Nov 28, 2017
    Configuration menu
    Copy the full SHA
    709ac3e View commit details
    Browse the repository at this point in the history

  3. added click event to show all the trips:

    @tvojnar
    tvojnar committed Nov 28, 2017
    Configuration menu
    Copy the full SHA
    f62a846 View commit details
    Browse the repository at this point in the history

  4. Merge pull request #1 from tvojnar/get-trips 

    Get trips
    @tvojnar
    tvojnar authored Nov 28, 2017
    Configuration menu
    Copy the full SHA
    6fd4dcc View commit details
    Browse the repository at this point in the history

Commits on Nov 29, 2017

  1. added data-id to each tr so I can access the id to get the trip detai… 

    …ls for a trip
    @tvojnar
    tvojnar committed Nov 29, 2017
    Configuration menu
    Copy the full SHA
    91598b9 View commit details
    Browse the repository at this point in the history

  2. can access new attributes for model after fetch for trip detauls

    @tvojnar
    tvojnar committed Nov 29, 2017
    Configuration menu
    Copy the full SHA
    6cab10e View commit details
    Browse the repository at this point in the history

  3. trying to figure out how to access all the attributes at once

    @tvojnar
    tvojnar committed Nov 29, 2017
    Configuration menu
    Copy the full SHA
    4310db0 View commit details
    Browse the repository at this point in the history

  4. trip details are prepended to the correct spot when a trip is clicked on

    @tvojnar
    tvojnar committed Nov 29, 2017
    Configuration menu
    Copy the full SHA
    d8084d3 View commit details
    Browse the repository at this point in the history

  5. empty out the trip-details article before adding another trips details

    @tvojnar
    tvojnar committed Nov 29, 2017
    Configuration menu
    Copy the full SHA
    a9a9692 View commit details
    Browse the repository at this point in the history

  6. now I can hide the trip details

    @tvojnar
    tvojnar committed Nov 29, 2017
    Configuration menu
    Copy the full SHA
    48c1bcc View commit details
    Browse the repository at this point in the history

  7. changed the click event for getting the trip details to be applied to… 

    … the tr as it is generated
    @tvojnar
    tvojnar committed Nov 29, 2017
    Configuration menu
    Copy the full SHA
    538d3ba View commit details
    Browse the repository at this point in the history

  8. changed functionality to only make the fetch api call for the model i… 

    …f it hasn't already made an api call for that trip
    @tvojnar
    tvojnar committed Nov 29, 2017
    Configuration menu
    Copy the full SHA
    42310c5 View commit details
    Browse the repository at this point in the history

  9. Merge pull request #2 from tvojnar/trip-details 

    Trip details
    @tvojnar
    tvojnar authored Nov 29, 2017
    Configuration menu
    Copy the full SHA
    313d2bc View commit details
    Browse the repository at this point in the history

Commits on Nov 30, 2017

  1. basic post set up

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    f3c818a View commit details
    Browse the repository at this point in the history

  2. actually posting now

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    a29cb93 View commit details
    Browse the repository at this point in the history

  3. form is removed if trip is saved

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    1e69e5c View commit details
    Browse the repository at this point in the history

  4. added handleValidationFailures function

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    2e24203 View commit details
    Browse the repository at this point in the history

  5. added validation check via an if statement in addTripHandler

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    7747b9a View commit details
    Browse the repository at this point in the history

  6. added validations to the Trip model

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    864bba9 View commit details
    Browse the repository at this point in the history

  7. added url to the model so now I can make a bad most request without t… 

    …riggering update
    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    ff3e163 View commit details
    Browse the repository at this point in the history

  8. moved where the form is placed down one lace

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    fc66b9f View commit details
    Browse the repository at this point in the history

  9. adding functionality to hide and show the add a trip button

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    5036473 View commit details
    Browse the repository at this point in the history

  10. added a cancel button for the add-trip frm

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    15d1dc8 View commit details
    Browse the repository at this point in the history

  11. Merge pull request #3 from tvojnar/add-a-trip 

    Add a trip
    @tvojnar
    tvojnar authored Nov 30, 2017
    Configuration menu
    Copy the full SHA
    28ef304 View commit details
    Browse the repository at this point in the history

  12. pushing so I can checkout a differnt branch

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    d21e399 View commit details
    Browse the repository at this point in the history

  13. added a form to reserve the trip

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    9c274df View commit details
    Browse the repository at this point in the history

  14. added click event for reserving a trip, but right now it just sends a… 

    …n allert
    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    0abfef0 View commit details
    Browse the repository at this point in the history

  15. attempted to make custom url for Reservation and wrote reserveTrip me… 

    …thod to try to make an api call to reserve a trip. Not working yet
    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    05df3d3 View commit details
    Browse the repository at this point in the history

  16. added some comments to help me tomorrow

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    6c3b0e3 View commit details
    Browse the repository at this point in the history

  17. you can reserve a trip now

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    9cac868 View commit details
    Browse the repository at this point in the history

  18. switched to using a template method to display the status messages

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    ae5ff97 View commit details
    Browse the repository at this point in the history

  19. added book css for status messages to style.css

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    4384c76 View commit details
    Browse the repository at this point in the history

  20. can remove status messages by clicking the x

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    bb94dfd View commit details
    Browse the repository at this point in the history

  21. added extra attribute to reportStatus to direct where to put the stat… 

    …us messages. Still works for error messages that go to the top of the page
    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    66e5871 View commit details
    Browse the repository at this point in the history

  22. undid addition of if else statement to reportStatus

    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    12b698d View commit details
    Browse the repository at this point in the history

  23. success status for reserving trip is now put below the h4 for the res… 

    …erve form
    @tvojnar
    tvojnar committed Nov 30, 2017
    Configuration menu
    Copy the full SHA
    f58b1e8 View commit details
    Browse the repository at this point in the history

Commits on Dec 1, 2017

  1. error messages from the server for the reserve trip are now added bel… 

    …ow the header above the form
    @tvojnar
    tvojnar committed Dec 1, 2017
    Configuration menu
    Copy the full SHA
    188906b View commit details
    Browse the repository at this point in the history

  2. client side validations work for the reservation now, and the error m… 

    …essages go to the right place
    @tvojnar
    tvojnar committed Dec 1, 2017
    Configuration menu
    Copy the full SHA
    484fecd View commit details
    Browse the repository at this point in the history

  3. done with adding trip functionality for now

    @tvojnar
    tvojnar committed Dec 1, 2017
    Configuration menu
    Copy the full SHA
    05bd4fa View commit details
    Browse the repository at this point in the history

  4. Merge pull request #4 from tvojnar/book-trip 

    Book trip
    @tvojnar
    tvojnar authored Dec 1, 2017
    Configuration menu
    Copy the full SHA
    367a803 View commit details
    Browse the repository at this point in the history

  5. fixed Trip so client side validations are now returned to be displaye… 

    …d to the page
    @tvojnar
    tvojnar committed Dec 1, 2017
    Configuration menu
    Copy the full SHA
    2aacfeb View commit details
    Browse the repository at this point in the history

  6. Merge pull request #5 from tvojnar/add-error-messages 

    fixed Trip so client side validations are now returned to be displaye…
    @tvojnar
    tvojnar authored Dec 1, 2017
    Configuration menu
    Copy the full SHA
    414ced7 View commit details
    Browse the repository at this point in the history

  7. added url to Trip model, but I didn't need it anymore because adding … 

    …client side validations fixed the problem of adding an invalid Trip
    @tvojnar
    tvojnar committed Dec 1, 2017
    Configuration menu
    Copy the full SHA
    c2d71b9 View commit details
    Browse the repository at this point in the history

  8. table is sorted when a th is clicked on

    @tvojnar
    tvojnar committed Dec 1, 2017
    Configuration menu
    Copy the full SHA
    c891fef View commit details
    Browse the repository at this point in the history

  9. added css to change th when you sort by th

    @tvojnar
    tvojnar committed Dec 1, 2017
    Configuration menu
    Copy the full SHA
    12b8c9b View commit details
    Browse the repository at this point in the history

  10. add form to filter trips

    @tvojnar
    tvojnar committed Dec 1, 2017
    Configuration menu
    Copy the full SHA
    32448b5 View commit details
    Browse the repository at this point in the history