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(ios): Keep the metadata of image, add GPS when take picture from Camera (CB-6708) #230

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

lxbyes
Copy link

@lxbyes lxbyes commented Aug 12, 2016

Platforms affected

What does this PR do?

What testing has been done on this change?

Checklist

  • ICLA has been signed and submitted to [email protected].
  • Reported an issue in the JIRA database
  • Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
  • Added automated test coverage as appropriate for this change.

@cordova-qa
Copy link

Cordova CI Build has completed successfully.

Commit - Link
Dashboard - Link

Builder Name Console Output Test Report Device Logs
Windows 8.1 Store Link Link Link
Windows 10 Store Link Link Link
Windows 8.1 Phone Link Link Link
iOS Link Link Link
Android Link Link Link

@lxbyes
Copy link
Author

lxbyes commented Aug 15, 2016

test passed

@cordova-qa
Copy link

Cordova CI Build has one or more failures.

Commit - Link
Dashboard - Link

Builder Name Console Output Test Report Device Logs
Windows 8.1 Store Link Link Link
Windows 10 Store Link Link Link
Windows 8.1 Phone Link Link Link
iOS Link Link Link
Android Link Link Link

@@ -240,13 +240,9 @@ - (void)alertView:(UIAlertView *)alertView clickedButtonAtIndex:(NSInteger)butto

- (void)repositionPopover:(CDVInvokedUrlCommand*)command
{
if (([[self pickerController] pickerPopoverController] != nil) && [[[self pickerController] pickerPopoverController] isPopoverVisible]) {
NSDictionary* options = [command argumentAtIndex:0 withDefault:nil];
Copy link
Member

Choose a reason for hiding this comment

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

Hi.
Thanks for the PR.

I think you forked you repo before this changes were made and didn't sync before sending your PR, but this code is important to fix another bug.
Can you make sure that your fork is in sync with the latest code on this repo and then send the PR with just the relevant code to fix the issue?

Copy link
Author

Choose a reason for hiding this comment

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

Hi.
I have sync code to my forks.

@lxbyes
Copy link
Author

lxbyes commented Sep 6, 2016

ICLA has been signed and re submitted to [email protected].

@cordova-qa
Copy link

Cordova CI Build has completed successfully.

Commit - Link
Dashboard - Link

Builder Name Console Output Test Report Device Logs
Windows 8.1 Store Link Link Link
Windows 10 Store Link Link Link
Windows 8.1 Phone Link Link Link
iOS Link Link Link
Android Link Link Link

@claudioviola
Copy link

Hi! Is it merged with master branch?
If I install with cordova plugin add cordova-plugin-camera@latest
do I find it?
Thank you!

@mattss
Copy link

mattss commented May 8, 2017

Is there any update on getting this merged?

@sandstrom
Copy link

sandstrom commented Sep 18, 2017

friendly ping @dhrumil162 😄

@tinomen
Copy link

tinomen commented Jan 30, 2018

What is needed to merge this?

@janpio
Copy link
Member

janpio commented Sep 16, 2018

There is a conflict in a file now unfortunately. Could you take care of this @lxbyes? Thanks.

@janpio janpio added the bug label Sep 17, 2018
@chintan13
Copy link

any update on this merge please?

@janpio janpio changed the title CB-6708: [iOS]keep the metadata of image, add GPS when take picture from Camera. feat(ios): Keep the metadata of image, add GPS when take picture from Camera (CB-6708) Jul 3, 2019
Copy link
Member

@janpio janpio left a comment

Choose a reason for hiding this comment

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

Hard merge conflict, open comments, bit chaotic code in general - this requires a proper review before being merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants