-
Notifications
You must be signed in to change notification settings - Fork 433
Added REST API and example #148
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
Conversation
This looks excellent! Nicely done! |
We can split it up into a Service in ./shared/users/user.service.ts later, otherwise looks great |
Good idea mate, I will sort that out later. I will make the table a bit prettier as well I think amd throw some extra angular validation on there as well maybe. |
What's the highest priority work for this do you think atm? |
I have AoT "basically" working, but I need to figure out one issue it's throwing that seems to be something overall going wrong with I wanted to get ng2-bootstrap with bootstrap4 & SASS (for themeing) in there, but feel like that might be a next step for after AoT is finished. Maybe some type of basic Authentication would be nice? ala #59? Also not sure if #149 is fixed, but guessing so. |
Cool I will look into these and try get something done |
The fonts for glyphicons weren't coming through bootstrap-sass, you can see the 404s in the browser, gotta figure that one out too. |
Hello mate,
I didn't mean to close the other pull request I was deleting some of my remote branches after merging back to master.
Anyway, let me know what you think of the changes.
Thanks.
Closes #122