-
Notifications
You must be signed in to change notification settings - Fork 11
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
Reconsider supporting usdws version 2. #625
Comments
I will be OOO next week 5-24-21 - 5-28-21 so @crolarlibertyva will be the POC for this issue until I get back. |
Hey, @lukemarkwordtlibertyits @crolarlibertyva, thanks for submitting the issue. I'd like to get a little more context.
That said, we've gotten this request a few times now, so we'll plan for a spike to try using USWDS 2.0 in Formation when we can. The end goal there will be to either plan the work to make the migration or write a decision document explaining in detail why we're staying on v1. |
Sorry for the delay. Just got back from PTO. We use formation in |
Ahhh, that's good to know, thanks! I agree, I'd rather resolve the issue at the source. As far as the upgrade to USWDS 2.x goes, though, it's a matter of effort vs value. The design system team is very small (just two engineers), so we have to be really careful to spend our effort on the work with the greatest impact. That spike will determine the effort and value so we can prioritize appropriately. (We'll be sure to factor resolving security vulnerabilities into the value. 👍 ) Thanks for filing this issue! |
Is your feature request related to a problem? Please describe.
From my understanding usdws 2 is not currently supported. It seems like in issue 504 it was determined to be unnecessary. The issue that I have found is that the latest versions of usdws 1 (v1.6.13) still has some security vulnerabilities.
Describe the solution you'd like
Support usdws version 2.
Describe alternatives you've considered
I cannot think of a way to get rid of vulnerabilities in usdws 1 without forcing them. I guess I could also create a ticket in the usdws repository to take care of these vulnerabilities, but they may still be in there for a reason (i.e. a breaking change).
Additional context
Currently I am getting errors when trying to run usdws 2 with the most recent
@department-of-veterans-affairs/formation
version.The text was updated successfully, but these errors were encountered: