Skip to content

Taisir Hassan - Shopify Technical Backend Challenge#291

Open
taisirhassan wants to merge 7 commits into
Shopify:mainfrom
taisirhassan:feature/implement-crud
Open

Taisir Hassan - Shopify Technical Backend Challenge#291
taisirhassan wants to merge 7 commits into
Shopify:mainfrom
taisirhassan:feature/implement-crud

Conversation

@taisirhassan

Copy link
Copy Markdown

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