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

Remove react scripts #1746

Merged
merged 6 commits into from
Mar 12, 2025
Merged

Remove react scripts #1746

merged 6 commits into from
Mar 12, 2025

Conversation

Mathias-a
Copy link
Contributor

@Mathias-a Mathias-a commented Mar 9, 2025

Starta med dette fordi Bugge meldte på oss, idk 🤷

Denne endte opp med å gjøre en del, her er TL;DR:

  • Vi bruker ikke react scripts, det er dated, så fjerna det
  • Ser ut til at Storybook v7 ikke likte det, så oppgraderte til v8
  • Ser React-modal har en bug, tror det henger sammen med react versjon, får ta det siden

@Mathias-a Mathias-a self-assigned this Mar 9, 2025
@Mathias-a Mathias-a requested a review from emilte as a code owner March 11, 2025 18:30
@Mathias-a Mathias-a force-pushed the mathias/remove-react-scripts branch from 194eaac to 2c2db0d Compare March 11, 2025 19:34
Copy link
Contributor

@Snorre98 Snorre98 left a comment

Choose a reason for hiding this comment

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

😮‍💨

@Mathias-a Mathias-a merged commit a88ce12 into master Mar 12, 2025
5 checks passed
@Mathias-a Mathias-a deleted the mathias/remove-react-scripts branch March 12, 2025 09:58
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.

2 participants