Skip to content
This repository has been archived by the owner on Oct 24, 2024. It is now read-only.

feat: [MCIL-23, MCIL-62] Added service, controller and test for update terminal #31

Merged
merged 11 commits into from
Apr 23, 2024
Merged

feat: [MCIL-23, MCIL-62] Added service, controller and test for update terminal #31

merged 11 commits into from
Apr 23, 2024

Conversation

RiccardoGiuliani
Copy link
Contributor

@RiccardoGiuliani RiccardoGiuliani commented Apr 12, 2024 •
edited
Loading

Description

Created a new controller and related service to modify and update a terminal.

List of Changes

Motivation and Context

How Has This Been Tested?

  • Unit Test Suite
  • Integration Test Suite
  • Api Tests
  • Load Tests

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@RiccardoGiuliani RiccardoGiuliani requested a review from a team as a code owner April 12, 2024 12:56
@TommasoLencioni TommasoLencioni changed the title chore: [MCIL-23, MCIL-62] Added service, controller and test for update terminal feat: [MCIL-23, MCIL-62] Added service, controller and test for update terminal Apr 12, 2024
@RiccardoGiuliani RiccardoGiuliani merged commit d5a28e6 into main Apr 23, 2024
6 checks passed
@RiccardoGiuliani RiccardoGiuliani deleted the MCIL-23-update-terminal branch April 23, 2024 13:20
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lucaconsalvi lucaconsalvi lucaconsalvi left review comments

@TommasoLencioni TommasoLencioni TommasoLencioni approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@RiccardoGiuliani @TommasoLencioni @lucaconsalvi