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

Meta: When should Fetch integration happen? #179

Closed
yoavweiss opened this issue Nov 28, 2018 · 1 comment
Closed

Meta: When should Fetch integration happen? #179

yoavweiss opened this issue Nov 28, 2018 · 1 comment

Comments

@yoavweiss
Copy link
Contributor

yoavweiss commented Nov 28, 2018

@annevk has commented on #178, saying that we should not postpone work that depends on Fetch integration, but rather tackle it immediately.

I think that we all recognize we have a lot of tech debt to pay here when it comes to Fetch integration. To that end, we have 2 options:

  1. Hold off L2 until all Fetch integration is complete. That's likely to delay publication by a long while, as we would add tests which current implementations are likely to fail due to current incompatibilities. That would also mean that L3 related features will be held off until all that work is complete.
  2. Publish L2 ASAP while resolving all tech debt unrelated to Fetch integration, and start work on L3 and Fetch integration immediately after that. At that point, L3 feature work and the merge of the spec with the NavigationTiming spec can happen in parallel.

I believe 2) is the shortest path to pay off our debt here, while enabling us to ship L2, which is long overdue.

@igrigorik
Copy link
Member

Based on previous group discussions, we agreed on (2) and reflected this in our new charter — see note on Nav Timing. Our goal is to address the last batch of outstanding L2 questions, freeze and lock the L2, and then immediately start work on L3 where we integrate RT+NT on top of Fetch.

Closing, since I believe we made an explicit decision on this one already, but if there is more to discuss here, please feel free to reopen.

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

No branches or pull requests

2 participants