Skip to content

Conversation

@ramsesmoreno
Copy link
Contributor

Sometimes you need the static files to be generated in a folder different from public/docs, so I added a config parameter to change it.

@rahulhaque
Copy link
Contributor

@ramsesmoreno Won't this break the /docs route? If you really want to access the static files from - suppose the root directory of your project, you can make symbolic link or junction for that.

@ramsesmoreno
Copy link
Contributor Author

Hi, thank you for check this.

Is the docs route also used when using the static generation? My change only affects that. My intention was to be able to generate the doc outside the public folder because, for example, I don't want the documentation to be public.

I'll make some tests and let you know.

@shalvah
Copy link
Contributor

shalvah commented Apr 12, 2020

@ramsesmoreno Won't this break the /docs route? If you really want to access the static files from - suppose the root directory of your project, you can make symbolic link or junction for that.

You shouldn't be using routing for static generation. For static generation, just open the HTML file in your browser and you're good.

Copy link
Contributor

@shalvah shalvah left a comment

Choose a reason for hiding this comment

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

Note that the structure of this config file will be changed somewhat in the upcoming v5.

@shalvah shalvah merged commit b35300b into mpociot:master Apr 12, 2020
@rahulhaque
Copy link
Contributor

@ramsesmoreno Won't this break the /docs route? If you really want to access the static files from - suppose the root directory of your project, you can make symbolic link or junction for that.

You shouldn't be using routing for static generation. For static generation, just open the HTML file in your browser and you're good.

Now I see, my server is configured to run index.* file by default and that's why I can access the /docs route to serve documentation. Thank you for clearing that up.

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