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

Gjør tavla mer klimavennlig! #529

Merged
merged 1 commit into from
Nov 12, 2021
Merged

Gjør tavla mer klimavennlig! #529

merged 1 commit into from
Nov 12, 2021

Conversation

Collatty
Copy link
Contributor

Leste meg litt opp på webpack og fant ut at devtool ikke er noe man trenger å ha i prod (som vi kanskje burde skjønt basert på navnet :Pp) og som jeg ikke ser noe behov for at vi har i prod.

En kjapp sjekk viser at det reduserer js-bundlene med ca 75% som for oss vil si ca 3MB. Google-søket mitt til kilder av ubekreftet kvalitet sier at én MB internett er ca 20g CO2. Antar vi rundt 200 sidenedlastinger per dag (fant ikke nøyaktig tall i firebase-konsollen på dette så det er basert på 60 (ekte tall) daglige brukere og antatt 3-4 lastinger per bruker) vil det si 200 * 20 * 3 * 365 / 1000 = 4380 kg CO2 spart per år. Det virket faktisk som en del så gjerne dobbeltsjekk matten min her.

Anyways, tror dette vil gjøre susen og, om det funker så bra som jeg tror det gjør, vil det også gjøre at tavla scorer bedre på en god del performance metrics.

Før:

image

Etter:

image

@Collatty Collatty requested a review from a team November 12, 2021 09:12
Copy link
Contributor

@magnusrand magnusrand left a comment

Choose a reason for hiding this comment

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

Veldig bra og miljøbevisst PR 🌿.
Får passe på at warningsene forsvinner i ekte Prod

Copy link
Contributor

@draperunner draperunner left a comment

Choose a reason for hiding this comment

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

👏

@Collatty Collatty merged commit ecc33a6 into master Nov 12, 2021
@draperunner draperunner deleted the reduce-carbon-footprint branch November 12, 2021 14:10
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.

4 participants