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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

UUID dep is out of date and throwing a warning on install #1106

Closed
alanbsmith opened this issue Jun 11, 2021 · 1 comment 路 Fixed by #1408
Closed

UUID dep is out of date and throwing a warning on install #1106

alanbsmith opened this issue Jun 11, 2021 · 1 comment 路 Fixed by #1408
Labels
5.x bug Something isn't working
Projects
Current Sprint (7/20 - 8/9)

Comments

@alanbsmith
Copy link
Member

馃悰 Bug Report

When I installed our packages, I got the following warning:

warning @workday/canvas-kit-react > uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.

To Reproduce

npx create-react-app test-app
cd test-app
yarn add @workday/canvas-kit-react

Expected Behavior

No warnings

Actual Results

Browser (if applicable)

n/a

Link to repl or repo (highly encouraged)

n/a
@alanbsmith alanbsmith added the bug Something isn't working label Jun 11, 2021
@alanbsmith alanbsmith added this to Open in Backlog via automation Jun 11, 2021
@project-bot project-bot bot moved this from Open to Medium in Backlog Jun 14, 2021
NicholasBoll added a commit to NicholasBoll/canvas-kit that referenced this issue Jan 5, 2022
@NicholasBoll
chore: Remove uuid and update unique id generation
c120a03 
Fixes: Workday#1311, Workday#1106, Workday#664, Workday#602
@NicholasBoll NicholasBoll mentioned this issue Jan 5, 2022
Merged
6 tasks
@NicholasBoll NicholasBoll linked a pull request Jan 5, 2022 that will close this issue
feat(common): Remove uuid and update unique id generation #1408
Merged
6 tasks
workday-canvas-kit pushed a commit that referenced this issue Jan 7, 2022
@NicholasBoll
feat(common): Remove uuid and update unique id generation (#1408)
9efce13 
Our current id generation has issues.

- It generates very long ids that add noise to the page
- It is possible to generate a blank id
- Id generation is slow
- Snapshot tests require mocking the `uuid` module

Fixes: #1311, #1106, #664, #602

[category:Hooks]

Release Note:
**NOTE for jest snapshots**: This change removes the `uuid` package and instead will generate a one-time client seed and then create auto-incrementing ids. This change will not break UI or automated UI tests. It will break snapshot tests however. Previously, the only way to get stable ids for snapshot tests was to mock the `uuid` module. This was an implementation detail. To make snapshots work again, add the following to your jest setup file:

```ts
import {setUniqueSeed, resetUniqueIdCount} from '@workday/canvas-kit-react/common';

beforeEach(() => {
  setUniqueSeed('a'); // force set the seed
  resetUniqueIdCount(); // reset the unique id count
});
```

This will ensure each Jest snapshot has ids that look like `a0` and `a1` and will be the same every time the snapshot is run. Do not use these methods in production though - it may lead to inaccessible applications due to IDREF collisions.
@NicholasBoll
Copy link
Member

Closed by #1408

Backlog automation moved this from Medium to Done Jan 10, 2022
@NicholasBoll NicholasBoll removed this from Done in Backlog Jan 10, 2022
@NicholasBoll NicholasBoll added this to To-do (This Sprint) in Current Sprint (7/20 - 8/9) via automation Jan 10, 2022
@NicholasBoll NicholasBoll moved this from To-do (This Sprint) to Done in Current Sprint (7/20 - 8/9) Jan 10, 2022
@jaclynjessup jaclynjessup removed the p:2 label Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
5.x bug Something isn't working
Projects
No open projects
Current Sprint (7/20 - 8/9)
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(common): Remove uuid and update unique id generation NicholasBoll/canvas-kit
4 participants
@NicholasBoll @jpante @alanbsmith @jaclynjessup