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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃搫 Update article theme for single project, consume dataUrl #68

Merged
merged 3 commits into from
May 11, 2023

Conversation

fwkoch
Copy link
Member

@fwkoch fwkoch commented May 4, 2023

This modifies article-theme so it is now intended for single-project-to-site configurations. Primarily it removes projectSlug from the url.

This also consumes dataUrl on links/xrefs so source mdast json data can be accessed directly from any provided link (rather than appending .json to the source url). This has nice implications around routing the index page for static HTML, i.e. the index page url can be simply / while the mdast is available at /index.json (or something equivalent...). This helps out with executablebooks/mystmd#188

@rowanc1
Copy link
Member

rowanc1 commented May 11, 2023

Ok, this is tested to work for backwards compatible sites as well as for the next version -- going to merge and deploy the themes.

@rowanc1 rowanc1 merged commit e8802ba into main May 11, 2023
1 check passed
@rowanc1 rowanc1 deleted the feat/single-article branch May 11, 2023 21:59
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.

None yet

2 participants