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

Update Hosts & Loaders + Fix Image issue #3242

Merged
merged 8 commits into from
Aug 18, 2020

Conversation

SamBroner
Copy link
Contributor

I took an opinionated approach on image storage based on this issue (gohugoio/hugo#1240)

Looking for feedback on the docs besides this.

@SamBroner SamBroner requested a review from tylerbutler August 18, 2020 11:40
@ghost ghost added the triage label Aug 18, 2020
@tylerbutler tylerbutler added area: host Host related issues area: loader Loader related issues documentation Improvements or additions to documentation labels Aug 18, 2020
@tylerbutler tylerbutler linked an issue Aug 18, 2020 that may be closed by this pull request
@@ -20,7 +20,7 @@ Objects can have state, which is managed by Distributed Data Structures (DDSes).
DDSes are used to distribute state to clients. Instead of centralizing merge logic in the
server, the server passes changes (aka operations or ops) to clients and the clients perform the merge.

![A diagram of the Fluid Framework architecture](./images/architecture.png)
![A diagram of the Fluid Framework architecture](/docs/concepts/architecture.png)
Copy link
Member

Choose a reason for hiding this comment

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

I fixed this in commit acd818e. I'd like to keep the images alongside the article they're referenced in, which we can do using Page Bundles.

Copy link
Member

Choose a reason for hiding this comment

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

I didn't notice that you were reusing the images in multiple articles. We may need something better long term but this is OK for now.

@tylerbutler tylerbutler merged commit 8fdf9cd into microsoft:docs/hugo Aug 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: host Host related issues area: loader Loader related issues documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hosts and Loaders concept article
3 participants