Skip to content

Conversation

nicolsss
Copy link
Contributor

@nicolsss nicolsss commented Dec 5, 2022

Description

  • When editing a project, if we select it as inactive, we get a 400 HTTP error. The message of the response is "Incorrect body"

Files changed

  • time_tracker/projects/_domain/project.py

Task board

Acceptance criteria

  • Given a TT admin, when they want to select a project as inactive, then the status code of the HTTP request should be 200.
  • Given a TT admin, when they want to select a project as inactive, then appear this message "The data has been saved successfully".

@github-actions
Copy link

github-actions bot commented Dec 5, 2022

[infrastructure/][stage] Terraform Plan 📖 failure

Show Plan ```

terraform

</details>
*Pusher: @nicolsss, Action: `pull_request`*

@github-actions
Copy link

github-actions bot commented Dec 5, 2022

[infrastructure/][prod] Terraform Plan 📖 failure

Show Plan ```

terraform

</details>
*Pusher: @nicolsss, Action: `pull_request`*

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 7, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@github-actions
Copy link

github-actions bot commented Dec 7, 2022

[infrastructure/][stage] Terraform Plan 📖 failure

Show Plan ```

terraform

</details>
*Pusher: @nicolsss, Action: `pull_request`*

@github-actions
Copy link

github-actions bot commented Dec 7, 2022

[infrastructure/][prod] Terraform Plan 📖 failure

Show Plan ```

terraform

</details>
*Pusher: @nicolsss, Action: `pull_request`*

@nicolsss nicolsss merged commit d71cd4f into master Dec 12, 2022
@nicolsss nicolsss deleted the TTL-727-bug-as-a-software-developer-i-want-to-fix-error-400-on-project-creation branch December 12, 2022 15:12
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.

2 participants