Skip to content

Feature/4725 web api persistence - #890

Open
sdavis-amido wants to merge 8 commits into
masterfrom
feature/4725-Web-API-Persistence
Open

Feature/4725 web api persistence#890
sdavis-amido wants to merge 8 commits into
masterfrom
feature/4725-Web-API-Persistence

Conversation

@sdavis-amido

Copy link
Copy Markdown
Contributor

4725 - Add CosmosDB persistence to the WebAPI project

📲 What

Add CosmosDB persistence to the WebAPI project to bring it in line with the other CQRS projects.

🤔 Why

For parity with CQRS projects.

🛠 How

Added CosmosDB implementation as per the CQRS projects.

👀 Evidence

N/A

🕵️ How to test

mvn clean test

✅ Acceptance criteria Checklist

  • Code peer reviewed?
  • Documentation has been updated to reflect the changes?
  • Passing all automated tests, including a successful deployment?
  • Passing any exploratory testing?
  • Rebased/merged with latest changes from development and re-tested?
  • Meeting the Coding Standards?

@sdavis-amido

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@sdavis-amido

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@sdavis-amido

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@sdavis-amido

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@sonarqubecloud

Copy link
Copy Markdown

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 1 Code Smell

91.3% 91.3% Coverage
0.0% 0.0% Duplication

Copilot AI review requested due to automatic review settings June 4, 2026 21:03
* @return List of MenuDTO
*/
List<Menu> findAllByRestaurantIdAndNameContaining(
UUID restaurantId, String searchTerm, Integer pageSize, Integer pageNumber);
* @return List of MenuDTO
*/
public List<Menu> findAllByNameContaining(
String searchTerm, Integer pageSize, Integer pageNumber);
* @param pageNumber page no
* @return List of MenuDTO
*/
public List<Menu> findAllByRestaurantId(UUID restaurantId, Integer pageSize, Integer pageNumber);
return new ResourceUpdatedResponse(categoryId);
}

public void delete(UUID menuId, UUID categoryId, String correlationId) {
private final UpdateCategoryMapper updateCategoryMapper;

public ResourceCreatedResponse create(
UUID menuId, @Valid CreateCategoryRequest body, String correlationId) {
return new ResourceUpdatedResponse(itemId);
}

public void delete(UUID menuId, UUID categoryId, UUID itemId, String correlationId) {
UUID categoryId,
UUID itemId,
@Valid UpdateItemRequest body,
String correlationId) {
}
}

public void delete(UUID menuId, String correlationId) {
UUID restaurantId,
Integer pageSize,
Integer pageNumber,
String correlationId) {

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

private final SearchMenuResultItemMapper searchMenuResultItemMapper;

public ResourceCreatedResponse create(@Valid CreateMenuRequest body, String correlationId) {
public MenuDTO create(@Valid CreateMenuRequest dto, String correlationId) {
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