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.

Sign up for GitHub

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

Jump to bottom

Not compatible with all versions of Catbox #7

Closed
niklasR opened this issue Jan 10, 2018 · 2 comments
Closed

Not compatible with all versions of Catbox #7

niklasR opened this issue Jan 10, 2018 · 2 comments
Assignees
@nspragg
Labels
enhancement

Comments

@niklasR
Copy link
Contributor

niklasR commented Jan 10, 2018 •
edited by nspragg
Loading

Tests fail with versions 8, 9 and 10 of Catbox.

Last working version was: 7.1.5
@nspragg nspragg changed the title Only works with specific versions of catbox. Not the latest. Not compatible with all versions of Catbox Jan 10, 2018
@nspragg nspragg added the bug label Jan 10, 2018
@nspragg nspragg self-assigned this Jan 10, 2018
@nspragg
Copy link
Contributor

nspragg commented Jan 10, 2018 •
edited
Loading

On initial inspection it seems catbox started making a migration to Node 8 (breaking changes) from version 8.0.0.

Further details pending but we'll most likely have to update our code to use async/await

@nspragg nspragg added enhancement and removed bug labels Jan 10, 2018
@nspragg
Copy link
Contributor

nspragg commented Feb 26, 2018

Fixed in version 2.0.0 - async/await upgrade.

@nspragg nspragg closed this as completed Feb 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nspragg nspragg

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@niklasR @nspragg