PabbleDabble commented on issue #679:
URL: 
https://github.com/apache/cordova-plugin-camera/issues/679#issuecomment-1655611206

   > Hi @PabbleDabble Would you consider creating a fork or a pull request with 
your changes on the original cordova-plugin-camera repository? This could 
really help others in the community.
   > 
   > Thanks for your contribution!
   
   
   https://github.com/PabbleDabble/cordova-plugin-camera
   
   Here's the steps I took after forking master from this main
   ```
   $ git clone https://github.com/PabbleDabble/cordova-plugin-camera.git
   $ cd cordova-plugin-camera
   $ git remote add a12-fork https://github.com/dongourdet/cordova-plugin-camera
   $ git remote add a13-fork https://github.com/jalios/cordova-plugin-camera
   $ git fetch --all
   # just a fast-forward
   $ git cherry-pick a13-fork/master
   # second pick had to resolve conflicts (just whitespace / confirm)
   $ git cherry-pick a12-fork/master
   $ git status
   $ git push
   ```
   
   For the record, I think the A12 cherry-pick is still probably not perfect as 
it asks for permission to delete the temp file after capture each time which is 
not great (but better than a crash). 
   
   Here's where I found comments / notes to why I did what I did...
   Link to A12 issue - 
https://github.com/apache/cordova-plugin-camera/issues/679
   Link to A13 issue - 
https://github.com/apache/cordova-plugin-camera/issues/826#issuecomment-1610030014


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to