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

New docs extras #1675

Merged
merged 5 commits into from
Jun 28, 2021
Merged

New docs extras #1675

merged 5 commits into from
Jun 28, 2021

Conversation

alexcjohnson
Copy link
Collaborator

Several updates to dash, designed to enable multi-language docs generated from markdown to work better:

  • 8503dfa - extra_hot_reload_paths: specify paths to be watched recursively for changes in any file, when you know that the app behavior depends on these files. I want this because I'm reading markdown files and converting them to Dash components.
  • 77c15c5 - provide more granular entry points to the component generator. I already have all the metadata.json( and package.json) files for all our component packages, I just want to use these to generate non-Python backends in order to read out the docstrings.
  • 9d0e821 - remove the constraint that requests_pathname_prefix ends with routes_pathname_prefix. This coincidentally just came up in Unable to load _dash-layout and _dash-dependencies from dash app running behind Nginx  #1665 but when you are serving your app behind a reverse proxy that rewrites URLs that constraint needs to be violated.

Contributor Checklist

  • I have added entry in the CHANGELOG.md

Copy link

@jdamiba jdamiba left a comment

Choose a reason for hiding this comment

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

These changes look good to me! 💃🏿

@alexcjohnson alexcjohnson merged commit c77912a into dev Jun 28, 2021
@alexcjohnson alexcjohnson deleted the new-docs-extras branch June 28, 2021 15:27
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