Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Add blog posting explaining browser-state plans. [ci skip] #4276

Merged
merged 1 commit into from
Sep 3, 2019

Conversation

pocmo
Copy link
Contributor

@pocmo pocmo commented Sep 2, 2019

I wanted to write this blog posting for quite a while. There's much more to say but this can be follow-up postings. :)

@pocmo pocmo added the 🕵️‍♀️ needs review PRs that need to be reviewed label Sep 2, 2019
@pocmo pocmo requested a review from a team as a code owner September 2, 2019 12:09
@pocmo pocmo changed the title Add blog posting explaining browser-state plans. Add blog posting explaining browser-state plans. [ci skip] Sep 3, 2019
@Amejia481 Amejia481 self-requested a review September 3, 2019 15:21
Copy link
Contributor

@csadilek csadilek left a comment

Choose a reason for hiding this comment

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

Nice!

docs/_posts/2019-05-23-deprecation copy.md Outdated Show resolved Hide resolved
docs/_posts/2019-05-23-deprecation copy.md Outdated Show resolved Hide resolved
docs/_posts/2019-05-23-deprecation copy.md Outdated Show resolved Hide resolved
Copy link
Contributor

@Amejia481 Amejia481 left a comment

Choose a reason for hiding this comment

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

👍

docs/_posts/2019-05-23-deprecation copy.md Outdated Show resolved Hide resolved
@pocmo
Copy link
Contributor Author

pocmo commented Sep 3, 2019

bors r=csadilek,Amejia481

@bors
Copy link

bors bot commented Sep 3, 2019

Merge conflict (retrying...)

bors bot pushed a commit that referenced this pull request Sep 3, 2019
4276: Add blog posting explaining browser-state plans. [ci skip] r=csadilek,Amejia481 a=pocmo

I wanted to write this blog posting for quite a while. There's much more to say but this can be follow-up postings. :)

4281: Move intent related code from browser-session to feature-intent. r=csadilek a=pocmo

In preparation for #4279 and #4257. :)



4294: Issue #4284: Add DownloadState to browser-state. r=csadilek a=pocmo

First steps for #4284. State is not synchronized between `browser-state` and `browser-session` yet. I need to figure out how to deal with the `Consumable<Download>`in `Session`. We already did that when we prototyped this earlier this year. I think that's the reason we introduced `Consumable.onConsume`.



Co-authored-by: Sebastian Kaspari <[email protected]>
@bors
Copy link

bors bot commented Sep 3, 2019

Build succeeded

@bors bors bot merged commit 5dc78f6 into mozilla-mobile:master Sep 3, 2019
@pocmo pocmo deleted the blog-state branch September 4, 2019 08:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🕵️‍♀️ needs review PRs that need to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants