popcodeorg / popcodeorg/popcode
Main bundle should only contain libraries needed for fresh environment load
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 191
- Forks
- 143
- Ø Merge
- 4 T. 11 Std.
- Gemergte PRs (30 T.)
- 5
Beschreibung
The main bundle should only contain the libraries needed to load a fresh environment, logged out, with no snapshot/gist ID in the URL and nothing in local storage. Everything else should be loaded via an async import().
Specific things that I don’t think we need:
- Firebase database: we need auth to determine login state but if you’re logged out, that should be the end of it
- GitHub
- Anything related to validations (we are mostly good here but I think the custom HTML validation module unintentionally pulls some of its dependencies into the main bundle)
- The loop breaker: should be easy to load async now that
compileProject()is async (and can skip it altogether if there is no JS)
We should also be deliberate about how we categorize various async bundles in the service worker configuration (e.g. I think the GitHub API should probably in a bundle classified as optional and loaded lazily)
Probably other stuff too, those are just some big ones that come to mind.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne am Einstiegspunkt des main-bundle und an der service-worker-Konfiguration und verfolge die Importe für Firebase, GitHub, Validierungen und den loop breaker. Bestätige, dass die frische, abgemeldete Umgebung nur ihre erforderlichen Bibliotheken lädt, während andere Funktionen asynchron geladen werden und Bundles wie die GitHub API für Lazy Loading kategorisiert werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- build-system, performance
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100