Skip to content

Conversation

@tylercraig9332
Copy link

@tylercraig9332 tylercraig9332 commented May 23, 2018

References issue #13

Note

This code isn't fully functioning. (More specifically the 3rd commit)-> So, you may ask why am I trying to add it to master -> Well, I won't be in for the rest of the week and I don't want all this code to be lost if one you guys decide to work on rendering shows into a view.

Things changed and added

I probably should have separated these as different pull requests.

  • Uninstall for boost works
  • Started working on creating a view for Shows
    • I figured a card system would look nice for this.
    • I however ran into a problem when I got to the template part. I didn't get it to render so it's mostly just code that's there but doesn't do anything yet -> This also means I couldn't test it.
    • I added a bit to the back-end factory class that would support this view.
  • Removed some template files that should have been removed before.
  • Finally, there are a few other react files that I didn't get around to populating that are just there.

@tylercraig9332 tylercraig9332 changed the title uninstall for boost and added some React files that will be populated once we start creating views uninstall for boost and added some React files that don't fully work yet May 23, 2018
@tylercraig9332 tylercraig9332 changed the title uninstall for boost and added some React files that don't fully work yet Uninstall for boost and added some React files May 23, 2018
@cambelem cambelem merged commit 0f06939 into AppStateESS:studentMaster May 29, 2018
@cambelem
Copy link

Not too thrilled with it not working, but it seems like a simple fix.

@tylercraig9332 tylercraig9332 mentioned this pull request May 30, 2018
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