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

Use max file size for validating attachments in Stream Dashboard #490

Merged

Conversation

laevandus
Copy link
Contributor

🎯 Goal

  • Use ChatClient.appSettings for validating attachment file sizes

🛠 Implementation

Consume LLC provided app settings when checking for max size.

🧪 Testing

N/A

☑️ Checklist

  • I have signed the Stream CLA (required)
  • Changelog is updated with client-facing changes
  • New code is covered by unit tests
  • Affected documentation updated (docusaurus, tutorial, CMS (task created)

@laevandus laevandus added enhancement New feature or request swiftui-repo labels May 21, 2024
@laevandus laevandus requested a review from a team as a code owner May 21, 2024 12:41
@laevandus laevandus force-pushed the feature/use-attachment-size-from-dashboard-app-settings branch 2 times, most recently from 03d2f00 to 3ef28c1 Compare May 22, 2024 11:30
@laevandus laevandus force-pushed the feature/use-attachment-size-from-dashboard-app-settings branch from 3ef28c1 to 9f1e086 Compare May 22, 2024 13:30
@laevandus laevandus enabled auto-merge (squash) May 22, 2024 13:53
Copy link

sonarcloud bot commented May 22, 2024

@laevandus laevandus merged commit 8e88fad into main May 22, 2024
10 checks passed
@laevandus laevandus deleted the feature/use-attachment-size-from-dashboard-app-settings branch May 22, 2024 14:22
@laevandus laevandus mentioned this pull request May 23, 2024
@martinmitrevski martinmitrevski mentioned this pull request Jun 6, 2024
@testableapple testableapple mentioned this pull request Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request swiftui-repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants