fix: refactored incoming request parameter name (#40231)
## Description - Fixes sync of remote branch in ce Fixes #40157 ## Automation /ok-to-test tags="@tag.Git" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!IMPORTANT] > 🟣 🟣 🟣 Your tests are running. > Tests running at: <https://github.com/appsmithorg/appsmith/actions/runs/14409483938> > Commit: 71bcdea29a412ad6be16507c95d74508c0d08fba > Workflow: `PR Automation test suite` > Tags: `@tag.Git` > Spec: `` > <hr>Fri, 11 Apr 2025 18:16:50 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Refactor** - Updated the internal naming convention used in Git operations for improved clarity and consistency. This change is entirely under-the-hood, ensuring that end-user functionality and default behaviors remain the same. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
This commit is contained in:
parent
0260c095c3
commit
28a629ff57
|
|
@ -249,11 +249,10 @@ public class GitApplicationControllerCE {
|
||||||
public Mono<ResponseDTO<List<GitRefDTO>>> getReferences(
|
public Mono<ResponseDTO<List<GitRefDTO>>> getReferences(
|
||||||
@PathVariable String branchedApplicationId,
|
@PathVariable String branchedApplicationId,
|
||||||
@RequestParam(required = false, defaultValue = "branch") RefType refType,
|
@RequestParam(required = false, defaultValue = "branch") RefType refType,
|
||||||
@RequestParam(required = false, defaultValue = "false") Boolean pruneBranches) {
|
@RequestParam(required = false, defaultValue = "false") Boolean pruneRefs) {
|
||||||
log.debug("Going to get branch list for application {}", branchedApplicationId);
|
log.debug("Going to get branch list for application {}", branchedApplicationId);
|
||||||
return centralGitService
|
return centralGitService
|
||||||
.listBranchForArtifact(
|
.listBranchForArtifact(branchedApplicationId, ARTIFACT_TYPE, BooleanUtils.isTrue(pruneRefs), GIT_TYPE)
|
||||||
branchedApplicationId, ARTIFACT_TYPE, BooleanUtils.isTrue(pruneBranches), GIT_TYPE)
|
|
||||||
.map(result -> new ResponseDTO<>(HttpStatus.OK, result));
|
.map(result -> new ResponseDTO<>(HttpStatus.OK, result));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user