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

Add support for multiple positions at the same employer #239

Closed
wants to merge 3 commits into from

Conversation

mekantor
Copy link

@mekantor mekantor commented Dec 3, 2024

When being with a single employer for a long time and progressing through titles, it may be desirable to show this.
Use the positions list to implement this, with the properties name, start_date, and end_date.
When templates detect positions, they will ignore position and prefer to iterate over this list.

@mekantor mekantor mentioned this pull request Dec 17, 2024
@sinaatalay
Copy link
Collaborator

Thank you! We recently decided to switch to Typst, therefore I think we shouldn't merge this. I am copying and pasting my response to another PR (#211), as it's for the same reason:

I think it would be the best to not merge this.

We are planning to transition to Typst soon, and each design update like this increases the complexity of maintenance. We need to ensure that design (LaTeX templates) works as expected with various combinations of design settings, which requires extensive manual testing. Currently, RenderCV runs robustly with its LaTeX templates, and we wouldn't want to break it right before we switch to Typst. Typst will require a lot of work. We can implement these in Typst.

For those interested in using these keys in RenderCV:

It is already quite simple to do so. That's why we allow users to use arbitrary keys in their YAML files and override the default LaTeX templates. Check out the links for details.

@sinaatalay sinaatalay closed this Dec 25, 2024
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