8000 Epic: tracking the removal of react · Issue #786 · ipfs/service-worker-gateway · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
Epic: tracking the removal of react #786
Open
@SgtPooki

Description

@SgtPooki

We want the service worker gateway to be super lightweight.. in order to do that we should remove react and make sure we have a very easy to understand and legible codebase that doesn't require transpilation or minification.

While this issue will likely be open for a long time, there are a large number of tasks that can be done to chip away at this goal:

  • no config page viewing on subdomains
  • 504, unknown-fetch-error, no-service-worker-error, and origin isolation warning pages should be in their own .html file in /public and not require react at all.
  • remove react routing that waits for react to be fully loaded to determine which page to render
  • All error pages (504, unknown error during fetch) should:
    1. unregister the service worker
    2. link to root domain config (it's on the landing page) to allow users to adjust things

I think we should still have react on the landing page because it will make development much more maintainable, but by completely separating the content retrieval and verification from the UI and react code, it will be much easier for users to verify and audit that the content they are getting is what they expect, without having to dive through a complex react ui codebase to do so.

This is mostly done with #773 and #782 (e.g. #785), but there is a lot of react that can be removed and migrated to lighter-weight javascript.

Related #773 #775 #778 #780 #782 #784 #785

cc @lidel @2color

Metadata

Metadata

Assignees

No one assigned

    Labels

    P1High: Likely tackled by core team if no one steps upeffort/weeksEstimated to take multiple weeksenhancementNew feature or requestexp/wizardExtensive knowledge (implications, ramifications) requiredkind/architectureCore architecture of projectkind/enhancementA net-new feature or improvement to an existing featurekind/maintenanceWork required to avoid breaking changes or harm to project's status quoneed/maintainers-inputNeeds input from the current maintainer(s)status/in-progressIn progress

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0