Skip to content

Conversation

@CharlesXu0488
Copy link
Contributor

@CharlesXu0488 CharlesXu0488 commented Oct 17, 2025

JIRA HCPSDKFIORIUIKIT-3001 requires for loading indicators for in-progress uploading attachments. To address that, 3 states, uploading, uploaded, and error, were introduced.

With this PR, in-grid/in-list loadings and errors are shown together with uploaded attachments. Additionally, extra info is allowed to be associated with each uploaded attachment. Apps need to provide default extra info value and use the extra info in custom style if the apps need the extra info.

BREAKING CHANGE: 🧨 data model

✅ Closes: 1

Uploading and Error
https://github.com/user-attachments/assets/3c615043-abcd-4ea8-8b0b-eaea6e93112a

Extra Info
https://github.com/user-attachments/assets/c3aba460-2020-41a9-a61f-0f24e9b8750a

Batch (change states outside of AttachmentGroup)
https://github.com/user-attachments/assets/267ea219-2d9e-4b04-938c-a5bac663e55e

Adding loading and error state to attachment grid/list; allowing extra
info for custom style

BREAKING CHANGE: 🧨 data model

✅ Closes: 1
@CharlesXu0488 CharlesXu0488 requested a review from a team as a code owner October 17, 2025 22:15
@CharlesXu0488 CharlesXu0488 requested review from billzhou0223 and dyongxu and removed request for a team October 17, 2025 22:15
Copy link
Contributor

@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.

  1. Add Public API Doc
  2. Add unit test

@CharlesXu0488 CharlesXu0488 requested a review from dyongxu October 20, 2025 19:38
@CharlesXu0488
Copy link
Contributor Author

Docs and Tests were added.

Copy link
Contributor

@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.

Correct some typos:
./_generated/StyleableComponents/Attachment/Attachment.generated.swift:37: attachement ==> attachment
./_generated/StyleableComponents/AttachmentButtonImage/AttachmentButtonImage.generated.swift:17: Attachements ==> Attachments
./_generated/StyleableComponents/AttachmentButtonImage/AttachmentButtonImage.generated.swift:32: poping ==> popping, pooping
./_generated/StyleableComponents/AttachmentButtonImage/AttachmentButtonImage.generated.swift:34: attachement ==> attachment
./_generated/StyleableComponents/AttachmentThumbnail/AttachmentThumbnail.generated.swift:24: attachement ==> attachment
./_generated/StyleableComponents/AttachmentGroup/AttachmentGroup.generated.swift:11: Attachements ==> Attachments
./_generated/StyleableComponents/AttachmentGroup/AttachmentGroup.generated.swift:30: maximium ==> maximum
./_generated/StyleableComponents/AttachmentGroup/AttachmentGroup.generated.swift:32: attachemnt ==> attachment
./_generated/StyleableComponents/AttachmentGroup/AttachmentGroup.generated.swift:34: attachement ==> attachment

@CharlesXu0488 CharlesXu0488 requested a review from dyongxu October 21, 2025 17:15
@CharlesXu0488
Copy link
Contributor Author

CharlesXu0488 commented Oct 21, 2025

attachement

more typos found addressed

@dyongxu dyongxu merged commit f540fee into SAP:main Oct 21, 2025
12 checks passed
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