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

Ports - Elle #25

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

Ports - Elle #25

wants to merge 18 commits into from

Commits on Jun 21, 2019

  1. Configuration menu
    Copy the full SHA
    10dcc00 View commit details
    Browse the repository at this point in the history

Commits on Jun 22, 2019

  1. Built the card components to display text and emoji. Built the board …

    …compnoent to render a list of cards from hard-coded data. Currently it will not display results that are missing fieldsbecause those are considered undefined. Need to fix this.
    dev-elle-up committed Jun 22, 2019
    Configuration menu
    Copy the full SHA
    adb1e1e View commit details
    Browse the repository at this point in the history
  2. Seems the undefined problem was already fixed. Cleaned up a few thing…

    …s and manually tested. App is working as expected.
    dev-elle-up committed Jun 22, 2019
    Configuration menu
    Copy the full SHA
    1bfed96 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    eef1315 View commit details
    Browse the repository at this point in the history
  4. Wrote componentDidMount function to do a get request from axios API w…

    …hen Board is mounted. Set state of cards to the get request result. Caught errors are lifted to App via a callback function for display in the header.
    dev-elle-up committed Jun 22, 2019
    Configuration menu
    Copy the full SHA
    0b616e6 View commit details
    Browse the repository at this point in the history
  5. Changed Board's render function to display cards from the axios get r…

    …equest response. Discovered error messages were not set up properly; fixed that as far as I can tell.
    dev-elle-up committed Jun 22, 2019
    Configuration menu
    Copy the full SHA
    4247922 View commit details
    Browse the repository at this point in the history
  6. More, hopefully final, changes to error messages. Added remove button…

    … to each card. Not yet functional. Updated PropTypes.
    dev-elle-up committed Jun 22, 2019
    Configuration menu
    Copy the full SHA
    10e6add View commit details
    Browse the repository at this point in the history
  7. More, hopefully final, changes to error messages. Added remove button…

    … to each card. Not yet functional. Updated PropTypes.
    dev-elle-up committed Jun 22, 2019
    Configuration menu
    Copy the full SHA
    aa8f18c View commit details
    Browse the repository at this point in the history

Commits on Jun 23, 2019

  1. Wrote removeCard function which makes an API call to delete the card …

    …from the database. Connected this to the button's onClick event handler. Need to make the view update.
    dev-elle-up committed Jun 23, 2019
    Configuration menu
    Copy the full SHA
    22074c3 View commit details
    Browse the repository at this point in the history
  2. Added to the removeCard function some logic to remove the card from t…

    …he view. Also cleaned up some code and comments. NOTE Right now the API call is commented out for deletions. This is to preserve data for ther rest of the app development. UNCOMMENT BEFORE DEPOLYMENT OR PROJECT SUBMISSION
    dev-elle-up committed Jun 23, 2019
    Configuration menu
    Copy the full SHA
    bf8906e View commit details
    Browse the repository at this point in the history
  3. Added form for adding a new card. Post request is working; card gets …

    …added to the database. Need to implement the update to the view. Do this with setState.
    dev-elle-up committed Jun 23, 2019
    Configuration menu
    Copy the full SHA
    fe248cc View commit details
    Browse the repository at this point in the history
  4. Broken. Error is Objects are not valid as a React child. found object…

    … with keys allCards. Making this branch to try a fix. Move rendering of cards to the render function in board. Save all responses from the API directly to the cards array in state.
    dev-elle-up committed Jun 23, 2019
    Configuration menu
    Copy the full SHA
    938ce13 View commit details
    Browse the repository at this point in the history
  5. Fixed things so that a new card will display. Fix: added a map to ini…

    …tial get request to pull the text and emoji data out of the card object. Now need to go through and fix delete and whatever else broke in the process.
    dev-elle-up committed Jun 23, 2019
    Configuration menu
    Copy the full SHA
    346c28f View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    447c69c View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    8aa24e6 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    febf2ff View commit details
    Browse the repository at this point in the history

Commits on Jun 24, 2019

  1. Configuration menu
    Copy the full SHA
    81c5bf6 View commit details
    Browse the repository at this point in the history
  2. Tests implement for Card and NewCardForm components. Snapshots create…

    …d and tests are passing.
    dev-elle-up committed Jun 24, 2019
    Configuration menu
    Copy the full SHA
    c42e326 View commit details
    Browse the repository at this point in the history