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

Document webserver deployment for #929 #932

Merged
merged 4 commits into from
Dec 9, 2021
Merged

Conversation

gavinband
Copy link
Contributor

Here's a first go at this - please use / edit as needed.

Copy link
Owner

@tlienart tlienart left a comment

Choose a reason for hiding this comment

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

Looks good to me just the __site__ to change for __site but otherwise good, thank you!


## Deploying on an existing webserver

The contents of the `__site__` folder can simply be deployed to a path on an existing server as follows.
Copy link
Owner

Choose a reason for hiding this comment

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

__site

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in eba6986

The contents of the `__site__` folder can simply be deployed to a path on an existing server as follows.
Supposing you wish your site to appear at `http://my.example.com/path/to/my/franklin/site/`. You would take the following steps:

* Prepare the `__site__` directory by running `optimize( prepath='/path/to/my/franklin/site/', minify = false )`.
Copy link
Owner

Choose a reason for hiding this comment

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

idem

* Prepare the `__site__` directory by running `optimize( prepath='/path/to/my/franklin/site/', minify = false )`.
Franklin.jl does not use relative links, so this step is needed to ensure that the links between site elements are correct. (The `minify = false` argument is optional but is currently recommended.)

* Copy the contents of the `__site__` directory to the target location using your chosen method.
Copy link
Owner

Choose a reason for hiding this comment

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

maybe here after method you could add (e.g. using rsync)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in eba6986

@tlienart tlienart merged commit 65c3bde into tlienart:master Dec 9, 2021
@tlienart
Copy link
Owner

tlienart commented Dec 9, 2021

Great, thanks!

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.

2 participants