Skip to content

chore: 🤖 [3155|3154]removed deprecated code#1559

Merged
dyongxu merged 6 commits intoSAP:mainfrom
KevinZK:jira_3155_main
Feb 25, 2026
Merged

chore: 🤖 [3155|3154]removed deprecated code#1559
dyongxu merged 6 commits intoSAP:mainfrom
KevinZK:jira_3155_main

Conversation

@KevinZK
Copy link
Copy Markdown
Collaborator

@KevinZK KevinZK commented Feb 10, 2026

SwiftUI: remove deprecated code for FilterFeedbackBarButtonModel, FilterFeedbackBarItemModel and SortViewModel

SwiftUI: remove deprecated code for FilterFeedbackBarButtonModel,
FilterFeedbackBarItemModel and SortViewModel
@KevinZK KevinZK requested a review from a team as a code owner February 10, 2026 02:33
@KevinZK KevinZK requested review from billzhou0223 and removed request for a team February 10, 2026 02:33
guard let nextValue = nextValue() else { return }
value = nextValue
}
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

File Length Violation: File should contain 700 lines or less: currently contains 1360 (file_length)

@dyongxu
Copy link
Copy Markdown
Collaborator

dyongxu commented Feb 10, 2026

/summarize

@dyongxu dyongxu requested a review from xiaoyu0722 February 10, 2026 22:38
}
*/
/// Option list picker configuration for styling
public struct OptionListPickerButtonConfiguration {
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.

Seems, these are used for deprecated components, I think we should remove them also.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

OptionListPickerButtonConfiguration used in new component, removed OptionListPickerItem

Copy link
Copy Markdown
Collaborator

@dyongxu dyongxu left a comment

Choose a reason for hiding this comment

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

resolve merge conflict

@KevinZK
Copy link
Copy Markdown
Collaborator Author

KevinZK commented Feb 25, 2026

Resolved conflicts

@KevinZK KevinZK requested a review from dyongxu February 25, 2026 06:52
@dyongxu dyongxu merged commit aa3a7e4 into SAP:main Feb 25, 2026
11 checks passed
@KevinZK KevinZK deleted the jira_3155_main branch February 26, 2026 02:09
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.

3 participants