Client side crud#7
Open
MTse3 wants to merge 32 commits into
Open
Conversation
adjusted gulpfile for css added working test for getAll(); almost working post test
minified css to be included in html adjusted html to containerize elements
adjusted some testing issues with build added some services that could potentially be implemented
player tests all pass again
changed gulpfile to remove concat all css files changed to scss scss files imported into app.scss index requires app.scss
attempted to adjust layout
There was a problem hiding this comment.
Two $scope.update methods? I don't think so.
added dummy directive issue with not being able to find template html file
crud is now in services cleaned up excess comments
made options for right left batting and throwing numbers input takes number value stylized title
set defaults for colors and button size/radius stylized button and heading using declared variables in base scss
added sample directives for transclude and forms file structure works, need to adjust for content commented out parts of html to potentially use as directive
added an player edit template for editing a player (can be removed after cancel button works) moved bulk of main into players_view html needs to fix issue with cancel button not showing
removed edit templates(redundant with form directive) fixed typo with player's position not showing
…button text to be create new player created table to display all the bio stats for the players in the database
Styles Css
more directives and table added
…ked_rest_api into client-side-crud
also merged directives (from sass branch) to be more updated
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
2 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.
angular crud with no auth
still need to edit cancel buttons