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

Missing themes #36

Open
thomasdavis opened this issue Nov 29, 2023 · 18 comments
Open

Missing themes #36

thomasdavis opened this issue Nov 29, 2023 · 18 comments

Comments

@thomasdavis
Copy link
Member

thomasdavis commented Nov 29, 2023

This is an dedicated to people who wish to request themes to be added to the hosted registry.

It is linked from the registry when a theme is not found when trying to render.

Note: recently we disabled a bunch of legacy themes from working with the registry. This will unfortunately/regrettably affect users using these themes. They were disabled because their latest versions were to out of date and concerned many critical errors in their code.

I will personally be keeping an eye an out for themes which are still being regularly used on the registry and fixing them as soon as possible.

If you were negatively affect, apologies again, please post the theme you want re-instated.

Currently looking into bringing back:

  • onepage
  • papirus
  • caffeine
@sachsgiri
Copy link

please bring back theme called "full"

@thomasdavis
Copy link
Member Author

@sachsgiri I've fixed up the full theme, it should work as it was now.

@clmay
Copy link

clmay commented Dec 2, 2023

Here's a vote for onepage! What all is involved in bringing a theme back "online"—are you just opening a PR with dependency updates, or are you forking them into the jsonresume org, or are you just re-enabling them from the registry side, or?

I'd be open to giving it a go on my own and save you some work, if that's worth it; or maybe it's trivial enough that it's not worth trying to explain and you just prefer to get it done. :)

@thomasdavis
Copy link
Member Author

Hey @clmay, I'm a bit busy at the moment so if you wanted to take a stab that would be awesome.

I am just bringing them into the repo, the onepage repo is quite simple -> https://github.com/ainsleyc/jsonresume-theme-onepage

If you just copy what I did here 7a56ff5

Which just downloads the repo into packages, then get rid of the readFileSync and wrap the styles and template in template literals and import directly.

And also bumping the version of handlebars to latest.

@levino
Copy link
Collaborator

levino commented Dec 2, 2023

We should add a section in the readme or add some other markdown file explaining the procedure and also highlighting the issues of vercel serverless deployment and node fs.

@ObserverOfTime
Copy link
Contributor

So why was my theme disabled? 🤔

@levino
Copy link
Collaborator

levino commented Dec 4, 2023

Dear @ObserverOfTime please provide a more complete description of your problem. Which theme is "yours"? How do you determine that it is "missing"?

@ObserverOfTime
Copy link
Contributor

#52

@miraldo
Copy link

miraldo commented Dec 5, 2023

It was fantastic to have OnePage back.

@thomasdavis
Copy link
Member Author

@ObserverOfTime thanks for pointing out that elegant does use fs. There is some strange problem I haven't figured out yet, but in short when Vercel deploys the code/themes to it's functionless environments, it doesn't seem to think that a lot of these files are for the bundle, so it skips them. Perhaps because that example is an assets folder it decides to include it. I will further investigate.

@miraldo Sorry about the inconvenience, I've just fixed and re-added the onepage theme to the repo (currently deploying)

@alexanderdavide
Copy link
Contributor

alexanderdavide commented Dec 6, 2023

Hey @thomasdavis, I'd love to have onepage-plus back. It made minor changes to onepage I really enjoyed.

I've opened a PR #59. 🚀

@thomasdavis
Copy link
Member Author

@skoenig
Copy link

skoenig commented Dec 30, 2023

Hey, would be nice if you could include the engineering theme.

@levino
Copy link
Collaborator

levino commented Dec 30, 2023

Just FYI, it is not so much about whether or not to include the themes or not. The missing themes are broken due to an outdated source and often a problematic usage of and assumptions about the local file system. So the themes need to be fixed and can only then be included.

@thomasdavis
Copy link
Member Author

@skoenig I've always loved that theme. Unfortunately the new system doesn't support the using of fs.readFileSync in npm templates.

You either need to use a build system or use import to get files in. I could help out writing a PR if you don't have the time. Otherwise checkout the packages/templates folder in this repo to see how other themes have migrated handlebars style themes.

@skoenig
Copy link

skoenig commented Jan 13, 2024

@thomasdavis Thanks for the hint about the templates, I'll try to find some time to work on making it compatible with the new system in the next few days. At the risk of overlooking something obvious: Is the complete code of a theme supposed to be vendored in this repository here under packages?

@Greg-Bush
Copy link

Hello! Could you please add my extension for kendall theme:
https://www.npmjs.com/package/jsonresume-theme-kendall-bushmanov

@kasperab
Copy link

I just published version 1.0.0 of my theme ”Vitae”, and I would like for it to be included in the registry. Thanks!

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

No branches or pull requests

10 participants