Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

feat(Storage): file removal ability. #75

Merged
merged 1 commit into from
Jul 17, 2016
Merged

feat(Storage): file removal ability. #75

merged 1 commit into from
Jul 17, 2016

Conversation

NathanWalker
Copy link
Contributor

Adds the ability to remove a file:

removeFile

You can pass in remote file path to remove it.

  firebase.removeFile({
    // optional, can also be passed during init() as 'storageBucket' param so we can cache it
    bucket: 'gs://n-plugin-test.appspot.com',
    // the full path of an existing file in your Firebase storage
    remoteFullPath: 'uploads/images/telerik-logo-uploaded.png'
  }).then(
      function () 
        console.log("File removed.");
      },
      function (error) {
        console.log("File removal Error: " + error);
      }
  );

Docs updated. Android/iOS.

@NathanWalker
Copy link
Contributor Author

This also dries up some code throughout all the file methods. 👍

@EddyVerbruggen
Copy link
Owner

Great work, Nathan!

I'll merge it and add the TS definition.

Thanks a bunch 👍

@EddyVerbruggen EddyVerbruggen merged commit d27609e into EddyVerbruggen:master Jul 17, 2016
EddyVerbruggen added a commit that referenced this pull request Jul 17, 2016
EddyVerbruggen added a commit that referenced this pull request Jul 17, 2016
- Added TS definition
- Fixed an issue on iOS where error was never set as the method only supports 1 param
EddyVerbruggen added a commit that referenced this pull request Apr 18, 2017
feat(Storage): file removal ability.
EddyVerbruggen added a commit that referenced this pull request Apr 18, 2017
EddyVerbruggen added a commit that referenced this pull request Apr 18, 2017
- Added TS definition
- Fixed an issue on iOS where error was never set as the method only supports 1 param
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants