Added upload multiple images from local storage feature#5
Added upload multiple images from local storage feature#5ganeshsp1 wants to merge 1 commit intoFirebaseExtended:masterfrom
Conversation
|
Updated Readme to correct service_account_key.json file name to service-account-key.json since the former causes an exception while uploading videos Hence found that the correct file name should be service-account-key.json as opposed to what was mentioned in the readme. |
|
Hi there, this is a very useful PR, can someone please merge it? |
prakhar1989
left a comment
There was a problem hiding this comment.
Regarding the second screenshot: can we use the app's theme colors instead of the blue?
| Navigator.pop(context); | ||
| }, | ||
| child: Text('OK'), | ||
| ) |
There was a problem hiding this comment.
Can this be a snackbar or a dialog instead? IMO, we can just navigate to the previous screen & show a snackbar informing user that images are being uploaded in the background.
| new FloatingActionButton.extended( | ||
| icon: const Icon(Icons.add_a_photo), | ||
| heroTag: 1, | ||
| label: const Text('Add Images'), |
There was a problem hiding this comment.
Having two buttons doesn't look that great :( Can we use a FAB menu or just use the icon?
| new FloatingActionButton.extended( | ||
| icon: const Icon(Icons.file_upload), | ||
| heroTag: 2, | ||
| label: const Text('Upload Images'), |
There was a problem hiding this comment.
Lets remove the text, and just use the icons? (as shown here)




Added an option to upload multiple images from the gallery, such that the user can click images before hand and upload them all together. The images will be uploaded in the background.
UI Change : New FloatingActionButton to Upload Images.
New Pub dependency Added :multi_media_picker - to select and upload multiple images.
Code Change has been integrated in the camera.dart file to resuse the code for uploading images to firebase storage