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

🥇 Inssue create the candidates section - Raquel #14

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

Conversation

RaquelLima7
Copy link

Pull Request Description:

This Pull Request added various functionalities and improvements to the project, aiming to make it more robust, scalable, and easy to maintain.

closes #12

Key Changes:

  • Addition of Candidate Model: Added a Candidate table to the database, containing fields such as name, date_of_birth, and email. The email field is unique, and the date of birth cannot be in the past.

  • Controller CRUD: Implemented a controller for CRUD operations. In the index route, it is now possible to pass parameters such as limit and offset, as well as filter the elements.

  • Helper for Filtering: Created a helper to facilitate the use of filtering in the index route, aiming to keep the application cleaner and more organized.

  • Model Validations: Added proper validations in the Candidate model.

  • Seed Data: In the seeds.rb file, added code to include 10 candidates in the database for testing purposes.

  • RSpec Tests: Wrote tests using RSpec. Additionally, a shared_examples folder was created to add reusable tests.

  • Addition of API V1 Routes: Added routes for API V1.

  • Swagger Documentation: Added Swagger documentation to facilitate understanding and usage of the API.

Objective:
The goal of this Pull Request is to enhance the quality and functionality of the project, following best software development practices and contributing to a more satisfactory user experience.

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.

Create the candidates section
1 participant