added my initial submission for iss44, does not have much functionality#59
Open
neilchokriwala wants to merge 8 commits intomasterfrom
Open
added my initial submission for iss44, does not have much functionality#59neilchokriwala wants to merge 8 commits intomasterfrom
neilchokriwala wants to merge 8 commits intomasterfrom
Conversation
Contributor
Author
|
The issue I am having is that I would like to see the gui button Delete. I can not see that. I also tried to add a sample message that will display in html. I am not sure which html address to put in as well. I |
Contributor
|
You need to enable your router in client/spa/js/main.js. An example of how to do that is in the comments. To see your module in action, you need to use localhost:3000/spa/index#students/1 (assuming that you have a student in your database with the id of 1. Try that and let us know how it goes. |
Contributor
Author
|
thanks Todd, I really appreciate it |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is not a complete version at all. I am trying to get coaches to review this, and try to help me understand what is wrong with this code as it is.