Skip to content

Commit

Permalink
fix(product): review fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
IvanMaidurov committed Aug 6, 2024
1 parent fdf020e commit c27334d
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,8 @@
* add config `commerce.checkout.orderService.skipCartValidation` to disable cart validation that happens right before placing an order in the OrderService

**search**
* fixed fake service to interpret sort options and gives away empty promotion
* **Breaking:** fixed fake service to interpret sort options and gives away empty promotion
* removed deprecation notice from fields `Asc` and `Desc` in `SortOption` struct

## v3.11.0

Expand Down
2 changes: 1 addition & 1 deletion product/module.go
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ commerce: {
if enabled {
jsonTestDataFolder?: string & !=""
jsonTestDataLiveSearch?: string & !=""
sorting: [SearchSorting, ...SearchSorting]
sorting: [...SearchSorting]
}
deliveryCodes: [...string] | *["testCode1", "testCode2"]
}
Expand Down
2 changes: 0 additions & 2 deletions search/domain/service.go
Original file line number Diff line number Diff line change
Expand Up @@ -63,10 +63,8 @@ type (
// SelectedDesc true if sorting by this field is active
SelectedDesc bool
// Asc - represents the field that is used to trigger ascending search.
// Deprecated: use "Field" and "SelectedAsc" instead to set which field should be sortable
Asc string // Should it be deprecated ?? marked as deprecated in 2019 and used in 2020
// Desc - represents the field that is used to trigger descending search.
// Deprecated: use "Field" and "SelectedDesc" instead to set which field should be sortable
Desc string // Should it be deprecated ?? marked as deprecated in 2019 and used in 2020
}

Expand Down

0 comments on commit c27334d

Please sign in to comment.