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

copypaste_advanced compatibility with jspreadsheet 11 #44

Open
randlet opened this issue Apr 22, 2024 · 4 comments
Open

copypaste_advanced compatibility with jspreadsheet 11 #44

randlet opened this issue Apr 22, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@randlet
Copy link

randlet commented Apr 22, 2024

Plugin Name with version

Copypaste Advanced 3.3.1

JSpreadsheet version

v10.x

Describe the bug

Jspreadsheet 11 is out now and it included some API changes which broke Advanced Copy Paste v3.3.1 initialization. In particular jspreadsheet.events is no longer available so the code at line 1008 here: https://www.npmjs.com/package/@jspreadsheet/copypaste_advanced?activeTab=code throws an exception.

As a workaround I was able to do jspreadsheet.events = { paste: jspreadsheet.paste } and things seem to work ok now.

Reproduction steps

1.  Initialize jspreadsheet version 11 with copypaste advanced plugin
2. Observe exception like:
: Cannot read properties of undefined (reading 'paste')
    at Function.<anonymous> (index.js:1008:1)
    at Object.t [as setPlugins] (index.js:122:52)
    at t.<computed> (index.js:123:1)
    at t.<computed> [as spreadsheet] (index.js:408:195)
    at index.js:522:196
@randlet randlet added the bug Something isn't working label Apr 22, 2024
@GBonnaire
Copy link
Owner

Hi,

Thanks to your signalement. I work on upgrade on my plugins to v11.

I will update package version limitation

Stay tuned here, i tell you when new version are available

@randlet
Copy link
Author

randlet commented Apr 24, 2024

Thanks! No rush :)

@kaushikd74
Copy link

@GBonnaire hey, we are looking for an update to this plugin as well :-)

@GBonnaire
Copy link
Owner

Hey.

thanks i will update soon. sorry for delay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants