Skip to content

Conversation

dennisreimann
Copy link
Collaborator

Fixes the build issues in #189.

@thunderbiscuit Can you check if this resolves the build error for you and if so merge this and then rebase your blog PR on top of the fix?

@ConorOkus updated the spiral theme and the latest version already includes the missing function. For reference: The theme is maintained here: https://github.com/spiralxyz/vuepress-devkit-theme

Copy link

netlify bot commented Aug 9, 2024

Deploy Preview for awesome-golick-685c88 ready!

Name Link
🔨 Latest commit a5317cc
🔍 Latest deploy log https://app.netlify.com/sites/awesome-golick-685c88/deploys/66b6694f6d1ab700083973b6
😎 Deploy Preview https://deploy-preview-190--awesome-golick-685c88.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@thunderbiscuit
Copy link
Member

Wow thanks for the super quick fix!

ACK a5317cc. I see why I couldn't find it; I'm not part of the spiralxyz org. All is good now!

@thunderbiscuit thunderbiscuit merged commit a5317cc into master Aug 9, 2024
5 checks passed
@thunderbiscuit
Copy link
Member

Ok actually it only worked locally because I had added the method to my local theme. When I remove the function it still doesn't build, even though my package-lock shows

"@spiralbtc/vuepress-devkit-theme": "^0.19.0",

@dennisreimann
Copy link
Collaborator Author

Did you run npm install to upgrade the dependencies locally?

@thunderbiscuit
Copy link
Member

I had not! Thanks. But actually even after doing that I get the same error. Both my package.json and package-lock.json use the "@spiralbtc/vuepress-devkit-theme": "^0.19.0", dependency.

It also appears that it's breaking for Netlify (#189).

@dennisreimann dennisreimann deleted the upgrade-deps branch August 10, 2024 06:35
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