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

fix(storage): Custom Prefix Android #2142

Merged
merged 2 commits into from
Oct 11, 2022
Merged

Conversation

fjnoyp
Copy link
Contributor

@fjnoyp fjnoyp commented Sep 20, 2022

Add missing case for public -> guest

Fix warnings in iOS

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@fjnoyp fjnoyp requested a review from HuiSF September 20, 2022 17:49
@fjnoyp fjnoyp requested a review from a team as a code owner September 20, 2022 17:49
@codecov-commenter
Copy link

codecov-commenter commented Sep 20, 2022

Codecov Report

Merging #2142 (73c95a9) into main (4839c47) will increase coverage by 0.28%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #2142      +/-   ##
==========================================
+ Coverage   45.94%   46.23%   +0.28%     
==========================================
  Files         359      364       +5     
  Lines       10871    10977     +106     
==========================================
+ Hits         4995     5075      +80     
- Misses       5876     5902      +26     
Flag Coverage Δ
android-unit-tests ∅ <ø> (∅)
flutter-unit-tests 35.89% <0.00%> (-0.01%) ⬇️
ios-unit-tests 89.16% <ø> (-0.68%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...lify_storage_s3/lib/method_channel_storage_s3.dart 60.00% <0.00%> (-0.49%) ⬇️
..._pinpoint_ios/example/ios/Runner/AppDelegate.swift 0.00% <0.00%> (ø)
...e/ios/unit_tests/MockAnalyticsCategoryPlugin.swift 15.38% <0.00%> (ø)
...ios/example/ios/unit_tests/AtomicResultTests.swift 87.75% <0.00%> (ø)
.../ios/unit_tests/amplify_flutter_exampleTests.swift 94.59% <0.00%> (ø)
...y_flutter_ios/example/ios/Runner/AppDelegate.swift 0.00% <0.00%> (ø)

Copy link
Contributor

@dnys1 dnys1 left a comment

Choose a reason for hiding this comment

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

Needs some integ tests before I can approve

@fjnoyp fjnoyp force-pushed the fix/storage_custom_prefix_android_fix branch 2 times, most recently from 44d9d32 to 73c95a9 Compare October 2, 2022 16:32
fjnoyp and others added 2 commits October 5, 2022 11:33
Add missing case for public -> guest

Fix warnings in iOS
…_s3.dart

Co-authored-by: Dillon Nys <24740863+dnys1@users.noreply.github.com>
@fjnoyp fjnoyp force-pushed the fix/storage_custom_prefix_android_fix branch 2 times, most recently from 7f74577 to 6535662 Compare October 5, 2022 18:34
@fjnoyp fjnoyp merged commit 2f7421d into main Oct 11, 2022
ragingsquirrel3 pushed a commit that referenced this pull request Oct 12, 2022
* fix(storage): Custom Prefix Android

Add missing case for public -> guest

Fix warnings in iOS

* Update packages/storage/amplify_storage_s3/lib/method_channel_storage_s3.dart

Co-authored-by: Dillon Nys <24740863+dnys1@users.noreply.github.com>
@dnys1 dnys1 deleted the fix/storage_custom_prefix_android_fix branch March 30, 2023 01:04
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.

4 participants