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

feat(gao-de-location): add plugin #2857

Merged
merged 4 commits into from Jan 4, 2019
Merged

feat(gao-de-location): add plugin #2857

merged 4 commits into from Jan 4, 2019

Conversation

waliu
Copy link
Contributor

@waliu waliu commented Dec 28, 2018

�Through this plug-in, you can get the local location. Reverse Address Coding

@danielsogl danielsogl self-requested a review December 28, 2018 15:27
@danielsogl danielsogl self-assigned this Dec 28, 2018
@danielsogl danielsogl added the target: patch This PR is targeted for the next patch release label Dec 28, 2018
@danielsogl danielsogl changed the title Add a gaode Map Location feat(gao-de-location): add plugin Dec 28, 2018
* @returns {Promise<any>} Returns a Promise that resolves with the success return, or rejects with an error.
*/
@Cordova()
getCurrentPosition(): Promise<any> {
Copy link
Owner

Choose a reason for hiding this comment

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

Please provide the correct typings as an interface.

*/
@Plugin({
pluginName: 'GaoDeLocation',
plugin: 'cordova-plugin-gaodelocation-chenyu', // npm package name, example: cordova-plugin-camera
Copy link
Owner

Choose a reason for hiding this comment

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

Remove the placeholder comments

/**
* @name Gao De Location
* @description
* Get the current location
Copy link
Owner

Choose a reason for hiding this comment

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

Please provide a better description. I'm not that familiar with Chinese so I don't know the difference between this plugin and the Cordova geolocation plugin.

Copy link
Contributor Author

@waliu waliu left a comment

Choose a reason for hiding this comment

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

The modification has been completed as required.

@waliu
Copy link
Contributor Author

waliu commented Jan 3, 2019

Do you need any other changes?

@danielsogl
Copy link
Owner

No its fine. I will merge all open PR's tomorrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants