-
Notifications
You must be signed in to change notification settings - Fork 38
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
Make learning materials page CRUD using the Rails Admin gem #87
Open
Nemwel-Boniface
wants to merge
11
commits into
nairuby:main
Choose a base branch
from
Nemwel-Boniface:make_learning_materials_page_crud
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Make learning materials page CRUD using the Rails Admin gem #87
Nemwel-Boniface
wants to merge
11
commits into
nairuby:main
from
Nemwel-Boniface:make_learning_materials_page_crud
Commits on Apr 15, 2024
-
Configuration menu - View commit details
-
Copy full SHA for e8acb37 - Browse repository at this point
Copy the full SHA e8acb37View commit details
Commits on Apr 16, 2024
-
- Create the learning materials resource - Migrate the learning materials resource to update the schema file
Configuration menu - View commit details
-
Copy full SHA for 8717427 - Browse repository at this point
Copy the full SHA 8717427View commit details -
- Update routes with the admin route and the learning materials routes - Add associations between User and Learning materials models - Create Learning materials controller with index action
Configuration menu - View commit details
-
Copy full SHA for a6d167f - Browse repository at this point
Copy the full SHA a6d167fView commit details -
- Add CSS variables to reduce the number of repetitions of color codes - Add the default thumbnail that will display when no image is uploaded when learning material is created - Create the learning materials index view listing all the currently available learning materials
Configuration menu - View commit details
-
Copy full SHA for 5a18fcc - Browse repository at this point
Copy the full SHA 5a18fccView commit details
Commits on Apr 17, 2024
-
- Update all links to the learning materials page with the correct updated one (learning_materials_path). - Add authorization rules to the rails admin config to only allow users with role "organization_admin" access the dashboard.
Configuration menu - View commit details
-
Copy full SHA for f0da3d4 - Browse repository at this point
Copy the full SHA f0da3d4View commit details -
- Setup rspec in the project for unit tests - Configured Capybara to work with rspec in the spec_helper.rb file for integration/ feature tests
Configuration menu - View commit details
-
Copy full SHA for 78753a1 - Browse repository at this point
Copy the full SHA 78753a1View commit details -
- Write integration test to the application - Update learning materials code moving logic to the controller moving it from the views
Configuration menu - View commit details
-
Copy full SHA for a2ddd4e - Browse repository at this point
Copy the full SHA a2ddd4eView commit details -
Configuration menu - View commit details
-
Copy full SHA for bc13a7f - Browse repository at this point
Copy the full SHA bc13a7fView commit details
Commits on Apr 18, 2024
-
- Add a section that lists the currently available learning materials for people who do not want to view the cards listed above - Add validations in the learning materials model for the title, description and link - Refactor error message localization in LearningMaterial model to improve code maintainability
Configuration menu - View commit details
-
Copy full SHA for f4e6447 - Browse repository at this point
Copy the full SHA f4e6447View commit details
Commits on Apr 23, 2024
-
- Use recommended snake case format for naming my variables. - Eagerload thumbnails for the Learning materials to avoid N + 1 queries.
Configuration menu - View commit details
-
Copy full SHA for a5b1b1d - Browse repository at this point
Copy the full SHA a5b1b1dView commit details
Commits on Apr 24, 2024
-
- Remove Motor admin from the project along with its config files - Test application to make sure it works still as required after removing motor admin
Configuration menu - View commit details
-
Copy full SHA for a944f54 - Browse repository at this point
Copy the full SHA a944f54View commit details
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.