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

dev: pass html-options from repo-config #27

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

CNLHC
Copy link
Contributor

@CNLHC CNLHC commented Mar 21, 2024

Currently, the underlying export function supports receiving a html-options map for configurating the published html but it seems that this cli has not supported this feature yet.

This PR makes the publish-spa cli read the hrml-options from the repo config(a.k.a logsq.edn) and enable the user to control the generated html.

After logseq/logseq#11149 being merged, user can also define extra scripts they want to include into the published html in their logseq.edn

@happylolonly
Copy link

@CNLHC will this help for #34 ?

@CNLHC
Copy link
Contributor Author

CNLHC commented Jul 31, 2024

@CNLHC will this help for #34 ?

Sure, I think this PR will also solve the #34 with a little change.

@happylolonly
Copy link

Cool, lets merge it!

@CNLHC
Copy link
Contributor Author

CNLHC commented Jul 31, 2024

Cool, lets merge it!

I can support using user defined meta info( head\title...) in next few days.

But I have no idea when this PR can be reviewed and merged 😿

@happylolonly
Copy link

happylolonly commented Jul 31, 2024

Cool, lets merge it!

I can support using user defined meta info( head\title...) in next few days.

But I have no idea when this PR can be reviewed and merged 😿

Who is owner and can merge? @logseq-cldwalker ?

Also would be good to have options for add more meta types, and scripts.
Trying https://github.com/cybercongress/cyber/blob/master/.github/workflows/publish.yml#L28 , but having some .sh errors for now

@happylolonly
Copy link

@logseq-cldwalker ?

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