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

Chenqian-rails-intern-assessment-CRUD-implementations #293

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

GloriaJingCQ
Copy link

Hi!✨ Thank you for your time reviewing my code. This is a really interesting assessment, thanks for preparing this. It's my first time learning and using Ruby on Rails framework and I love learning through building! I worked more with node.js and django before. This assessment made me think about the selection of frameworks(their pros and cons) as well. I had a lot fun!

Changes Made:

  • Implemented CRUD actions, user can create, read, update, delete articles.
  • Implemented search feature, and can be easily extended based on future requirements. Users can type input in UI and search for title and content or more if needed.
  • Implemented user friendly UI for testing functionalities, applied bootstrap.
  • All tests passed

Features Implemented(Demo):

  • Create
    2024-01-29 20 13 39
  • Read
    Show a list of articles in main page. Also we can review the detail of each article
    2024-01-29 20 10 46
  • Update
    2024-01-29 20 11 27
  • Delete
    Needs confirmation before deletion
    2024-01-29 20 11 48
  • Search
    User can input the search key. Based on the test, search is on title and content. But can be easily extended to other fields.
    2024-01-29 20 13 05

Future Improvements:

  • Constraints check for each field. Considering the database's column requirements and other aspects, handle users' invalid input.
  • Cache can be applied to reduce unnecessary pressure on our database.
  • Advanced filter by field can be applied.
  • User login
  • ...

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