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

feat: 🎸 [JIRA: HCPSDKFIORIUIKIT-2708] avatar stack support #788

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

xiaoyu0722
Copy link
Contributor

grouped horizontal
Simulator Screenshot - iPhone 15 Pro Max - 2024-09-03 at 17 04 52 Simulator Screenshot - iPhone 15 Pro Max - 2024-09-03 at 17 05 00

@xiaoyu0722 xiaoyu0722 requested a review from a team as a code owner September 3, 2024 09:06
@xiaoyu0722 xiaoyu0722 requested review from billzhou0223 and removed request for a team September 3, 2024 09:06
@xiaoyu0722 xiaoyu0722 force-pushed the feat-avatar-stack-enhancement branch 2 times, most recently from 1066003 to 1f58254 Compare September 4, 2024 00:28
@xiaoyu0722 xiaoyu0722 force-pushed the feat-avatar-stack-enhancement branch from 1f58254 to b854caa Compare September 5, 2024 06:56
@xiaoyu0722 xiaoyu0722 force-pushed the feat-avatar-stack-enhancement branch from b854caa to 517c44e Compare September 9, 2024 00:46
@xiaoyu0722 xiaoyu0722 force-pushed the feat-avatar-stack-enhancement branch from 517c44e to 6c7376d Compare September 9, 2024 00:56

/// :nodoc:
var body: some View {
if self.count == 0 {
Copy link

Choose a reason for hiding this comment

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

Empty Count Violation: Prefer checking isEmpty over comparing count to zero. (empty_count)

@billzhou0223 billzhou0223 merged commit 910198e into SAP:main Sep 9, 2024
12 checks passed
@xiaoyu0722 xiaoyu0722 deleted the feat-avatar-stack-enhancement branch September 9, 2024 23:21
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