This repository has been archived by the owner on Nov 2, 2018. It is now read-only.
Courses render #83
Merged
Courses render #83
Conversation
This file contains 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
Created UI for forms List and Views Need to added finishing styles, refactorings, and transactions
- Finished Refactoring structure to separate structure from content - Finished connecting and testing transactional end-point modelling - Need to Finish styling some custom rendering - Need to isolate UI code into proper structure
We are now using webpack for all locally develop JS assets. This is because turbolink/sprockets is being difficult and not letting us declare templates after the fact. Follow the guideline in mount.js and also import the correct dependencies and webpack will take care of the rest. Next task: add more to the control-component stylings
We are now using webpack for all locally develop JS assets. This is because turbolink/sprockets is being difficult and not letting us declare templates after the fact. Follow the guideline in mount.js and also import the correct dependencies and webpack will take care of the rest. Next task: add more to the control-component stylings
…ly from CSRF tags. API tokens no longer necessary. Also styles material shadows onto course views. I'm going to add some padding, then move onto authorization schema and removing API tokens.
…to redirects with a page reload but instead a subsequent ajax to the redirected route we can no longer use redirects (without having to define specific API routes to redirect to)
Remaining tasks - Dynamic render views - Client pagination - Client filtering - API Limit
Working on search and paging
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Issue Addressed: Courses UI
Changes:
This merge adds in the full applet allowing users to create, view, edit, delete, and search through courses. Not that this doesn't yet include file uploads at the time of writing
Reviewer: @Jimmy_Lin