Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: static data in mocks #1574

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

Merged
merged 3 commits into from
May 19, 2022
Merged

fix: static data in mocks #1574

merged 3 commits into from
May 19, 2022

Conversation

greyscaled
Copy link
Contributor

No description provided.

@greyscaled greyscaled requested a review from a team as a code owner May 18, 2022 20:01
Copy link
Member

@kylecarbs kylecarbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will break Storybook. Time ago will always be changing! Is something broken rn?

@greyscaled
Copy link
Contributor Author

This will break Storybook. Time ago will always be changing! Is something broken rn?

This fixes Storybook, not breaks it.

To understand why not have dynamic new Date()s in our mocks, see that @Emyrk 's PR that touches only the backend here:

#1569

is causing a changeset in chromatic: https://www.chromatic.com/test?appId=624de63c6aacee003aa84340&id=628544a245901b004586ba74

@kylecarbs
Copy link
Member

@vapurrmaid I believe this will fix that page, but break the Workspaces page timestamps in Storybook.

@kylecarbs
Copy link
Member

@vapurrmaid it's probably best if Storybook page instances decide to use static vs. dynamic time themselves. Because there could be requirements to display ago on one page, and a specific timestamp on another, it seems like it'll be hard to standardize.

@greyscaled greyscaled marked this pull request as draft May 19, 2022 01:25
@greyscaled greyscaled marked this pull request as ready for review May 19, 2022 15:16
@greyscaled greyscaled merged commit e0165c5 into main May 19, 2022
@greyscaled greyscaled deleted the vapurrmaid/static-mock-data branch May 19, 2022 15:36
kylecarbs pushed a commit that referenced this pull request Jun 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants