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

Update archive NPM package #1930

Closed
canstudios-louisem opened this issue Mar 22, 2018 · 2 comments
Closed

Update archive NPM package #1930

canstudios-louisem opened this issue Mar 22, 2018 · 2 comments
Assignees
@canstudios-louisem
Labels
D: easy Straightforward issues, small time commitment S: merged Completed, reviewed, and merged issues T: enhancement Adding additional functionality
Milestone
0.5.0 Import/Export

Comments

@canstudios-louisem
Copy link
Contributor

canstudios-louisem commented Mar 22, 2018 •
edited
Loading

The archive dependency is massively out of date we should update it.
@canstudios-louisem canstudios-louisem added T: bug D: easy Straightforward issues, small time commitment labels Mar 22, 2018
@canstudios-louisem canstudios-louisem self-assigned this Mar 22, 2018
@canstudios-louisem canstudios-louisem added the S: awaiting-review Completed issues waiting on reviews label Mar 22, 2018
@canstudios-louisem canstudios-louisem mentioned this issue Mar 22, 2018
Merged
@canstudios-louisem canstudios-louisem added the I: critical High impact bugs, breaking functionality for the end-user - should be fixed ASAP label Mar 22, 2018
@tomgreenfield
Copy link
Contributor

Hi @canstudios-louisem, I'm struggling to replicate this – is there anything I'm missing?

Environment in case you're wondering is:

  • Chrome 65 on Windows 10
  • Node v6.11.5
  • develop commit 2656cf2

@canstudios-louisem
Copy link
Contributor Author

I should probably update this issue I think it was caused by an npm package that had been updated when it shouldn't have.

Its less critical than when i first thought but we should probably update that dependency anyway as its soooooooooooo old.

@canstudios-louisem canstudios-louisem added T: enhancement Adding additional functionality and removed I: critical High impact bugs, breaking functionality for the end-user - should be fixed ASAP T: bug labels Mar 23, 2018
@canstudios-louisem canstudios-louisem changed the title Export broken on develop Update archive NPM package Mar 23, 2018
@canstudios-louisem canstudios-louisem added S: merged Completed, reviewed, and merged issues and removed S: awaiting-review Completed issues waiting on reviews labels Apr 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@canstudios-louisem canstudios-louisem

Labels
D: easy Straightforward issues, small time commitment S: merged Completed, reviewed, and merged issues T: enhancement Adding additional functionality
Projects
None yet
Milestone
0.5.0 Import/Export
Development

No branches or pull requests
2 participants
@tomgreenfield @canstudios-louisem