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.

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

[SSDK-544] Replace supportSBS binary with expandable ApiType enum #162

Merged
merged 3 commits into from
Feb 5, 2024

Conversation

aokj4ck
Copy link
Contributor

@aokj4ck aokj4ck commented Jan 25, 2024

Description

Fixes SSDK-544

  • Change AbstractSearchEngine.init supportSBS: Bool = false parameter to apiType: ApiType = .SBS.
  • This changes the default API engine for discover/category and other API requests to SBS.
  • Add ApiType enum to represent non-Autofill and non-PlaceAutocomplete SearchEngine API types.
  • This will be expanded to include the new search-box API type

Checklist

  • Update CHANGELOG

@aokj4ck aokj4ck marked this pull request as ready for review January 26, 2024 15:46
@aokj4ck aokj4ck requested review from a team as code owners January 26, 2024 15:46
ghost
ghost previously approved these changes Jan 26, 2024
case .geocoding:
return .geocoding
case .SBS:
return .SBS

Choose a reason for hiding this comment

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

what is the significance of the "." in ".SBS"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just Swift syntax for referencing an enum case, and the type of the enum is inferred from the return value of the function's signature.

@aokj4ck aokj4ck requested review from mapbox-danny and a user February 2, 2024 20:02
@aokj4ck aokj4ck merged commit 742cf38 into main Feb 5, 2024
4 checks passed
@aokj4ck aokj4ck deleted the SSDK-544-add-public-api-type-and-engine-support branch February 5, 2024 19:15
aokj4ck added a commit that referenced this pull request Mar 20, 2024
…h arguments com.mapbox.mapboxsearch.enableSBS

- Removed in beta.33
- More thoroughly parameterized in 2.0.0-rc.2 / SSDK-544 / PR #162
aokj4ck added a commit that referenced this pull request Mar 20, 2024
…h arguments com.mapbox.mapboxsearch.enableSBS (#195)

### Description
- Removed in beta.33
- More thoroughly parameterized in 2.0.0-rc.2 / SSDK-544 / PR #162

#### Screenshots

| Before | After |
| -- | -- |
| <img width="1254" alt="Screenshot 2024-03-20 at 13 47 09" src="https://github.com/mapbox/mapbox-search-ios/assets/384288/ddd96362-00d9-4dba-b08a-697be1389124"> | <img width="1254" alt="Screenshot 2024-03-20 at 13 46 21" src="https://github.com/mapbox/mapbox-search-ios/assets/384288/86be713e-5d44-4fe2-bca7-c102ba78a0d6"> |
aokj4ck added a commit that referenced this pull request Apr 5, 2024
…h arguments com.mapbox.mapboxsearch.enableSBS (#195)

### Description
- Removed in beta.33
- More thoroughly parameterized in 2.0.0-rc.2 / SSDK-544 / PR #162

#### Screenshots

| Before | After |
| -- | -- |
| <img width="1254" alt="Screenshot 2024-03-20 at 13 47 09" src="https://github.com/mapbox/mapbox-search-ios/assets/384288/ddd96362-00d9-4dba-b08a-697be1389124"> | <img width="1254" alt="Screenshot 2024-03-20 at 13 46 21" src="https://github.com/mapbox/mapbox-search-ios/assets/384288/86be713e-5d44-4fe2-bca7-c102ba78a0d6"> |
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