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

Publish minutes of 2021-12-09 meeting #141

Merged
merged 1 commit into from
Dec 16, 2021
Merged

Publish minutes of 2021-12-09 meeting #141

merged 1 commit into from
Dec 16, 2021

Conversation

Rob--W
Copy link
Member

@Rob--W Rob--W commented Dec 9, 2021

Generated from https://docs.google.com/document/d/1QkwhEMtMS67JBUkl_WVPZ4lRSKoWcQNlLJSf_GwSXg8/edit using the tool and process from #105.

During this meeting we discussed or mentioned #134, #120, #93 and #118. Although added to the agenda, we did not have enough time to discuss #113 and #115.

@nifgraup
Copy link

Was #133 not on the agenda?

@Rob--W
Copy link
Member Author

Rob--W commented Dec 10, 2021

Was #133 not on the agenda?

Apparently not. But we didn't have enough time to get through all points that were on the agenda.

But #133 is about functionality that's lost if background pages were to be replaced with a service worker with a limited lifetime.
That broader topic was covered extensively during the meeting, as part of issue #134, #120 and the "Random Q&A related to MV3".

Note: if you'd like to discuss the specifics of #133, just post in #133; it's not necessary to wait for the video meetings to participate in the discussion.

* [tomislav] Disagree with viewpoint, it makes transitioning harder. Our plans are to support APIs in MV2 where possible.
* [timothy] Safari also has that philosophy. We'll also support service workers in MV2.
* [devlin] For this reason we support service workers and DNR in MV2 in Chrome. Some other APIs have a less drastic migration and are more utilities than dramatic replacement features, e.g. the action API replaces the browserAction API and is not supported in MV2 because migration is straightforward.
* [rob] Firefox develops new features restricted to MV3,
Copy link
Member

Choose a reason for hiding this comment

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

@Rob--W, looks like we only got the first half of your thought. Do you recall what the second half was?

[Issue 93](https://github.com/w3c/webextensions/issues/93): Support i18n.getMessage API in serviceWorker

* [carlos] Simeon mentioned in the past that there are issues with this method that may require design changes. Can you clarify?
* [devlin] An issue with the implementation is that it's synchronous. [spoke to Chrome implementation details related to process responsibility and message passing]
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* [devlin] An issue with the implementation is that it's synchronous. [spoke to Chrome implementation details related to process responsibility and message passing]
* [devlin] An issue with the implementation is that it's synchronous. [spoke to Chrome implementation details related to process responsibility, message passing between processes, challenges in supporting synchronous responses, and use cases that currently require synchronous responses]

I don't recall Devlin's comment with much specific detail, but I think this captures the essence of what he said a little more accurately.

@Rob--W
Copy link
Member Author

Rob--W commented Dec 14, 2021

Copy link
Member

@dotproto dotproto left a comment

Choose a reason for hiding this comment

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

Thanks for the updates, @Rob--W. I think we're good to go :)

@Rob--W Rob--W merged commit 51bfbae into main Dec 16, 2021
github-actions bot added a commit that referenced this pull request Dec 16, 2021
SHA: 51bfbae
Reason: push, by @Rob--W

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants