-
Notifications
You must be signed in to change notification settings - Fork 5
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
Allow output from 1clickbom to be pasted into tool #2
Comments
I think this could be expanded to allow copy and paste bi-directionally. @SDBowen, can't assign you yet (need to invite you to join the org) but can you take on this one? |
Yeah, will work on this in the next few days.
…On Sat, Jun 15, 2019, 1:31 PM Kaspar Emanuel ***@***.***> wrote:
I think this could be expanded to allow copy and paste bi-directionally.
@SDBowen <https://github.com/SDBowen>, can't assign you yet (need to
invite you to join the org) but can you take on this one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2?email_source=notifications&email_token=AFNJVHOVNF6W7NJYEKR6OS3P2THKFA5CNFSM4GCRY4G2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXYWF4Q#issuecomment-502358770>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFNJVHNUYWD2OETQX7T2HJTP2THKFANCNFSM4GCRY4GQ>
.
|
I was remembering it wrong during the call today, @kasbah. It is the copy function you already had in the code, allowing the user to copy the bom from the bom-builder to their clipboard. https://github.com/kitspace/bom-builder/blob/dev/src/App.js#L116 Since the 1-click-bom extension already has a copy function, implementing a paste from clipboard into the bom-builder would be the easiest way to getting this functionality. I left off trying to see how the 'Open' an existing bom functionality works, and seeing if I could do something similar to achieve this paste functionality. Also, I am unable to push a working branch: |
I had to change the base permission for the kitspace group. You should have write access to all repositories now. Can you push your branch? |
Yes, that worked. Branch is here: https://github.com/kitspace/bom-builder/tree/1ClickBom-copy-paste |
Ok to video chat in 10 mins? |
This was the first thing I tried to do, as I knew it had all the same columns, and I wanted to know what the features were :)
The text was updated successfully, but these errors were encountered: