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

doc: Do not override assets while building the styleguide #6408

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Jan 21, 2025

The output directory needs to be different from the input, otherwise the assets are overwritten and we can not use them (e.g. for the avatar).

This needs adjustments in Netlify to use styleguide/build for deployment instead of just styleguide. @juliusknorr

@susnux susnux added the feature: documentation Related to the documentation label Jan 21, 2025
@juliusknorr

This comment was marked as resolved.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
Copy link
Contributor Author

susnux commented Jan 21, 2025

I changed and rebased to see if this works, but have an error reported:

It was a typo by me... Fixed!

@juliusknorr juliusknorr merged commit 420139f into master Jan 21, 2025
19 checks passed
@juliusknorr juliusknorr deleted the fix/styleguide branch January 21, 2025 11:42
@raimund-schluessler
Copy link
Contributor

I guess this needs a backport to next? The styleguide for v9.0.0 does not work anymore https://next--nextcloud-vue-components.netlify.app/

@susnux
Copy link
Contributor Author

susnux commented Jan 21, 2025

/backport to next

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: documentation Related to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants