fix: ssr: remove memoizeone dep and replace it with local function in locale

EightfoldAI / octuple

The Octuple component library

MIT License

16 stars 45 forks source link

fix: ssr: remove memoizeone dep and replace it with local function in locale #811

Closed dkilgore-eightfold closed 2 months ago

dkilgore-eightfold commented 2 months ago

SUMMARY:

memoize-one dep contains a line referencing globalThis and falls back to global and self. This was incompatible with SSR, so I replaced it with logic that accomplishes the same memoization goal.

JIRA TASK (Eightfold Employees Only):

ENG-85282

CHANGE TYPE:

[x] Bugfix Pull Request
[ ] Feature Pull Request

TEST PLAN:

Pull the PR branch and run yarn and yarn storybook. Verify the ConfigProvider Locale story behaves as expected.

codesandbox-ci[bot] commented 2 months ago

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.