Skip to content
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

Taisir Hassan - Shopify Technical Backend Challenge #291

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

taisirhassan
Copy link

Pull Request: Implementing CRUD Functionality and Search Functionality for an Encyclopedia

Demo

Screen.Recording.2024-01-29.at.11.24.11.PM.mov

Description

Major Changes:

  • Implemented the Article model with title and content attributes.
  • Created a controller for articles with index, show, new, edit, create, update, and destroy actions.
  • Created views for each of the controller actions.
  • Styled all views with Bootstrap for a better user interface.
  • Added error handling for form submissions.
  • Documented all the code.

Elaboration on Changes Made:

Models

  • Added a create method in the Article model to allow article creation.
  • Added validations to ensure data integrity.

Controllers

  • Created new routes and actions (index, show, new, create, edit, update, destroy) for articles.
  • Implemented CRUD actions for articles.
  • Added a search action and implemented search functionality to find articles by title or content.
  • Implemented error handling and flash messages for a better user experience.

Views

  • Created new views (index, show, new, edit) for articles.
  • Updated existing views for consistency and user experience, including listing articles and displaying single articles.
  • Implemented forms and templates for creating, editing, and viewing articles.

Tests

  • Verified that all tests pass successfully.
  • Added comments and explanatory notes in the code where necessary to improve code readability.
Screenshot 2024-01-29 at 11 16 40 PM

Documentation

  • Added comments and explanatory notes in the code where necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant