-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Add Base64 plugin #1645
Add Base64 plugin #1645
Conversation
/** | ||
* This function encodes base64 of any file | ||
* | ||
* @param {filePath} Absolute file path |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change this line to @param filePath {string} Absolute file path
plugin: 'com-badrit-base64', | ||
pluginRef: 'window.plugins.Base64', | ||
repo: 'https://github.com/hazemhagrass/phonegap-base64', | ||
install: 'ionic cordova plugin add com-badrit-base64', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this line
plugin: 'com-badrit-base64', | ||
pluginRef: 'window.plugins.Base64', | ||
repo: 'https://github.com/hazemhagrass/phonegap-base64', | ||
install: 'ionic cordova plugin add com-badrit-base64', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This install
line is not needed since the plugin doesn't require any variables.
@Plugin({ | ||
pluginName: 'Base64', | ||
plugin: 'com-badrit-base64', | ||
pluginRef: 'window.plugins.Base64', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no need to have window.
here... plugins.Base64
is sufficient
/** | ||
* This function encodes base64 of any file | ||
* | ||
* @param {filePath} Absolute file path |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line should read:
* @param {string} filePath Absolute file path
|
||
/** | ||
* This function encodes base64 of any file | ||
* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove blank JSDoc line here
No description provided.