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.

Sign up for GitHub

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

Jump to bottom

[MODORDSTOR-435] Introduce new batch update pieces endpoint #463

Merged
merged 8 commits into from
Dec 23, 2024
Merged

[MODORDSTOR-435] Introduce new batch update pieces endpoint #463

merged 8 commits into from
Dec 23, 2024

Conversation

Saba-Zedginidze-EPAM
Copy link
Contributor

@Saba-Zedginidze-EPAM Saba-Zedginidze-EPAM commented Dec 18, 2024 •
edited
Loading

Purpose

[MODORDSTOR-435] Introduce new batch update pieces endpoint

Approach

  • Add new endpoint
  • Minor refactoring of PiecesAPI, move certain methods to a service class

@Saba-Zedginidze-EPAM Saba-Zedginidze-EPAM changed the title Modordstor 435 [MODORDSTOR-435] Introduce new batch update pieces endpoint Dec 18, 2024
@Saba-Zedginidze-EPAM Saba-Zedginidze-EPAM force-pushed the MODORDSTOR-435 branch 9 times, most recently from 5c2faa4 to 56dc6aa Compare December 18, 2024 15:58
@Saba-Zedginidze-EPAM Saba-Zedginidze-EPAM marked this pull request as draft December 19, 2024 09:38
@Saba-Zedginidze-EPAM Saba-Zedginidze-EPAM marked this pull request as ready for review December 19, 2024 12:39
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
80.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Saba-Zedginidze-EPAM Saba-Zedginidze-EPAM requested a review from a team December 19, 2024 13:42
@azizbekxm
Copy link
Contributor

image

@azizbekxm azizbekxm merged commit 5b528d3 into master Dec 23, 2024
8 checks passed
@azizbekxm azizbekxm deleted the MODORDSTOR-435 branch December 23, 2024 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BKadirkhodjaev BKadirkhodjaev BKadirkhodjaev approved these changes

@azizbekxm azizbekxm azizbekxm approved these changes

@SerhiiNosko SerhiiNosko SerhiiNosko 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.
4 participants
@Saba-Zedginidze-EPAM @azizbekxm @SerhiiNosko @BKadirkhodjaev