search

mitchelloharawild / vitae

R Markdown Résumés and CVs
https://pkg.mitchelloharawild.com/vitae/
1.19k stars 229 forks source link

moderncv won't compile tex file #176

Closed meflynn closed 2 years ago

meflynn commented 2 years ago

There may be an obvious solution to this, but when I try to compile a moderncv file with any of its themes I consistently get the following error:

! LaTeX Error: Command \researcheridsocialsymbol undefined.

I thought it might be triggered by the social arguments in the YAML but this happens even with the most bare bones YAML/file setup. and no other content. I've looked through the style files, too, but I'm not sure if this is an issue with an sty file or the tex template. Thanks for your help!

title: CV output: vitae::moderncv: theme: "banking"

meflynn commented 2 years ago

I dug into the tex template too but couldn't find any calls to this particular command.

meflynn commented 2 years ago

I resolved the issue. The moderncv.cls file was missing several social media/networking symbol commands under the "% other symbols" section. I've pasted the list below that resolved the issue. Note that the first one appears to be a spelling variation on one that was already listed.

\newcommand{\researcheridsocialsymbol}{} \newcommand{\googlescholarsocialsymbol}{} \newcommand{\telegramsocialsymbol}{} \newcommand{\whatsappsocialsymbol}{} \newcommand{\signalsocialsymbol}{} \newcommand{\matrixsocialsymbol}{} \newcommand*{\bornsymbol}{}

marthinsen commented 2 years ago

I have the same problem. I looks to me like a general issue. ​Should this issue be reopened?

The moderncv template test fails on both Windows and Linux for me. But your suggested changes fixes it. Do you mind if I make a PR out of it?

meflynn commented 2 years ago

I think I accidentally closed this prematurely. Probably a good idea to reopen for the developer to address. Sounds good to me!

mitchelloharawild commented 2 years ago

Thanks for the fix @meflynn, and for @marthinsen for the PR.

meflynn commented 2 years ago

Happy to help! Love the package.

On Aug 19, 2021, at 9:06 PM, mitchelloharawild @.***> wrote:

 Thanks for the fix @meflynn, and for @marthinsen for the PR.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.