Skip to content

Feat/125/create a new sprint#157

Merged
Kimoo193 merged 4 commits into
mainfrom
feat/125/create-a-new-sprint
Apr 17, 2025
Merged

Feat/125/create a new sprint#157
Kimoo193 merged 4 commits into
mainfrom
feat/125/create-a-new-sprint

Conversation

@Kimoo193
Copy link
Copy Markdown
Contributor

PR Checklist (required)

Please check if your PR fulfills the following requirements:

  • The commit message follows our guidelines.

  • Tests for the changes have been added (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation changes
  • Other... Please describe:

Related Issue

Closes #125

@Kimoo193 Kimoo193 added the feature For new feature label Apr 17, 2025
@Kimoo193 Kimoo193 requested a review from Copilot April 17, 2025 18:40
@Kimoo193 Kimoo193 self-assigned this Apr 17, 2025
@Kimoo193 Kimoo193 linked an issue Apr 17, 2025 that may be closed by this pull request
@Kimoo193
Copy link
Copy Markdown
Contributor Author

This pull request introduces functionality for sprint management, including creating sprints, validating sprint data, and integrating sprint-related routes into the application. The most important changes involve the addition of a new controller for sprint creation, route definitions, input validation, and updates to the server configuration.

Sprint Management Features:

  • Sprint Controller (src/controllers/sprint.controller.js):

    • Added a createSprint method to handle the creation of sprints, including validations for parameters, permissions, and sprint dates. It also calculates sprint status and ensures no overlapping sprints.
  • Sprint Validation (src/validations/sprint.validation.js):

    • Introduced comprehensive validation schemas using Joi for creating, updating, and managing sprint data, ensuring robust input validation.
  • Sprint Routes (src/routes/sprint.routes.js):

    • Defined a new route for creating sprints (POST /api/organization/:organizationId/team/:teamId/project/:projectId/sprint/create) with authentication middleware.

Server Configuration Updates:

  • Route Integration (src/index.js):

    • Imported and registered sprintRoutes in the main server file to enable sprint-related API endpoints. [1] [2]
  • CORS Policy Update (src/index.js):

    • Modified the CORS configuration to allow all origins (origin: '*') while retaining support for cookies.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 26 out of 26 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

src/validations/sprint.validation.js:3

  • [nitpick] Consider renaming 'sprintvalidation' to 'sprintValidation' to follow camelCase naming conventions for consistency.
export const sprintvalidation = (obj) => {

Comment thread src/index.js
@Kimoo193 Kimoo193 force-pushed the feat/125/create-a-new-sprint branch from f645c27 to 0adaafd Compare April 17, 2025 18:54
@Kimoo193 Kimoo193 merged commit 0f295fc into main Apr 17, 2025
5 checks passed
@Kimoo193 Kimoo193 deleted the feat/125/create-a-new-sprint branch April 17, 2025 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature For new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create a new sprint

2 participants