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

Change p5 dependency version to minimum 1.4.1 #318

Closed
taliacotton opened this issue Feb 16, 2024 Discussed in #317 · 9 comments · Fixed by #319
Closed

Change p5 dependency version to minimum 1.4.1 #318

taliacotton opened this issue Feb 16, 2024 Discussed in #317 · 9 comments · Fixed by #319
Assignees
Labels
dependencies Pull requests that update a dependency file enhancement good first issue

Comments

@taliacotton
Copy link

Discussed in #317

Originally posted by taliacotton February 15, 2024
I've noticed the component depends on a specific version of p5 (as of right now I see 1.9.0). I was wondering if there's a way to control which version of p5 it uses. The reason for this is that there are certain features of p5 in prior versions, so I'm wondering if it's possible to downgrade the version. Thanks!

// package-lock.json
    "node_modules/@p5-wrapper/react": {
      "version": "4.3.2",
      "resolved": "https://registry.npmjs.org/@p5-wrapper/react/-/react-4.3.2.tgz",
      "integrity": "sha512-m7MoL63grVVyu39czEi3V3tz29VM4zzd4K9MjNWZ5t7jb4itgHpw6FCiSG2JzU1SqElKgo9ny8NliavPWYYdrw==",
      "dependencies": {
        "microdiff": "^1.3.2",
        "p5": "^1.9.0"
      },
      "peerDependencies": {
        "react": ">= 18.2.0",
        "react-dom": ">= 18.2.0"
      }
    },
```</div>
@jamesrweb
Copy link
Collaborator

What specifically is 1.9 missing that you need from 1.4? The main point is security updates, lowering the version would open the package and consumers of the package up to CVEs which are patched in the latest versions.

@taliacotton
Copy link
Author

Makes total sense. There are a couple bugs in the current p5 version that you can see here and here, which they've flagged. As a temporary fix since I'm working on a time-sensitive project that depends on that update, I was hoping to revert back to an older version of p5... but I'm unable to do that too 😅 Hence my request.
Thank you!

@jamesrweb
Copy link
Collaborator

I see your comments in those threads, indeed these are issues but do the workarounds suggested such as pixel density not work for your needs? If not, I'll consider making p5 a peer dependency but that would entail a new major release since it would disrupt existing user installs and also the other aforementioned topics would need documented and so on too.

@taliacotton
Copy link
Author

I appreciate that, and I wouldn't be asking if I didn't try a dozen other things first, including all of their workaround suggestions and then some. Totally understand treating it as a major release. Thought I'd try!

@jamesrweb
Copy link
Collaborator

jamesrweb commented Feb 16, 2024

Give me this evening (it's 8pm in Germany right now) to think about it, you'll know my decision by the morning (German time) 😜

CC @yevdyko

@taliacotton
Copy link
Author

omg💖

@yevdyko
Copy link
Collaborator

yevdyko commented Feb 16, 2024

@jamesrweb I left some suggestions in the Discussions thread on how to avoid using a peer dependency

@yevdyko yevdyko self-assigned this Feb 17, 2024
@jamesrweb
Copy link
Collaborator

Version 4.3.3 is now released and you should be able to request that version of p5 now @taliacotton. Let us know if it works for you or not and we can either close this issue or work on a fix then.

CC @yevdyko

@jamesrweb jamesrweb added enhancement dependencies Pull requests that update a dependency file good first issue labels Feb 19, 2024
@jamesrweb jamesrweb linked a pull request Feb 19, 2024 that will close this issue
@jamesrweb
Copy link
Collaborator

Closing issue due to inactivity, @taliacotton - please comment here again if you have any issues with the latest deployed version and I can reopen this issue again. Otherwise I hope your problem is solved and good luck with your project 🍀😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement good first issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants