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

How to represent present work? #199

Closed
chrisdotcode opened this issue Nov 12, 2015 · 26 comments
Closed

How to represent present work? #199

chrisdotcode opened this issue Nov 12, 2015 · 26 comments

Comments

@chrisdotcode
Copy link
Member

cf. jsonresume/resume-cli#60

The above is a discussion of how to specify present work. In short, there are three proposed solutions:

  • Remove the endDate field entirely.
  • Make endDate null.
  • Make endDate an empty string.

I'm with the third one, and I definitely do not think that the field should be removed entirely if the work is still ongoing.

Themes can be left to the prettify the empty string.

@chrisdotcode chrisdotcode changed the title An empty string endDate should specify present work How to represent present work? Nov 12, 2015
@chrisdotcode chrisdotcode added this to the v1.0.0 milestone Nov 12, 2015
@aloisdg
Copy link
Contributor

aloisdg commented Nov 15, 2015

I have no problem with all of them, but I prefer the second one aka null.

If today is between my startDate and my endDate, then it is my current job. If endDate cant be determined (absent, null or empty) and startDate is before today, then it is my current job.

I can have 2 or 3 jobs, past jobs, current jobs and future jobs.

@stp-ip
Copy link
Member

stp-ip commented Nov 16, 2015

I would favor the 2 one considering the usual practices with json.

@chrisdotcode
Copy link
Member Author

So in #198, @KOLANICH suggests removing the field entirely, and replacing it all with a date array. My addition to that was that we can therefore represent presentWork with a boolean if we choose that solution.

Thoughts?

@stp-ip
Copy link
Member

stp-ip commented Nov 16, 2015

Would the usage of current as a boolean be too meaningless? Would in my opinion make everything more consistent, when used for every work, education etc. object.

@chrisdotcode
Copy link
Member Author

I like current, and present, and you're right in that they need to convey meaning.

IMO, current will be fine.

@olivif
Copy link
Collaborator

olivif commented Nov 18, 2015

I agree with @stp-ip here, having a flag for things that are on-going could be very useful and applicable to other objects not just education. Use cases I can think of are current jobs, current education, current projects, current publications (you can have the paper written but not published, in which case you might want to mark it as current). Same for things like events you are planning, volunteering work.

@chrisdotcode
Copy link
Member Author

We've actually resolved this problem in another thread, #198.

Present work takes cares of itself with the proposed solution.

@stp-ip
Copy link
Member

stp-ip commented Nov 18, 2015

@chrisdotcode in most cases yes, but to focus on the bibjson case. They don't have a current/not published state. We could use the year from bibJSON as we would other future dates as discussed in #198, but that needs to be a special thing to look at.

@chrisdotcode
Copy link
Member Author

To be frank, if it's not published... it's not published, and therefore shouldn't need to be noted.

As per another suggestion somebody else made in another thread, if we do use bibJSON for publications, we should use that as a black box format in which we follow their conventions for their necessary fields.

@stp-ip
Copy link
Member

stp-ip commented Nov 18, 2015

@chrisdotcode I agree with the convention and would not add a special string or so, but if their date string is in the future themes should know how to handle that and therefore show "current" most likely.

@beshur
Copy link

beshur commented Sep 26, 2016

So no decision here?
Should I update endDate every day for the resume to stay valid?

@aloisdg
Copy link
Contributor

aloisdg commented Sep 26, 2016

@beshur It is a long talk. Read more here: #198

@fartbagxp
Copy link

Mmm, I've been following this problem for about five months now and there hasn't been any resolution yet.

My workaround currently:
Convert the PDF to .docx with Adobe PDF (far superior than other PDF readers in terms of copying and pasting PDF) and appending the "Present" in myself before printing it out or converting it back to PDF.

@aloisdg
Copy link
Contributor

aloisdg commented Sep 29, 2016

Mmm, I've been following this problem for about five months now and there hasn't been any resolution yet.

Feel free to provide arguments to unlock the situation or a better solution directly. If you have time and motivation to help, it is more than welcome.

@thomasdavis
Copy link
Member

My vote is for option 2)

Taking types into consideration, an empty string would represent something else. (terrible explanation)

null is less verbose because we wouldn't need to repeat redundant properties in each section.

What is the official majority vote at this point, we can probably merge something in.

@mudassir0909
Copy link
Member

mudassir0909 commented Oct 12, 2016

I'm with null for representing "present work". And it's upto the themes to ensure they render "Present", "Current" or whatever the label when they encounter null in the endDate.

@aloisdg
Copy link
Contributor

aloisdg commented Oct 12, 2016

Before arguing about a solution, please read #198 too.

@alienkidmj12
Copy link

how do we input job history, if i have more than 1 job in there it reports that the current job doesnt have an end date :(

@noraj
Copy link

noraj commented Dec 31, 2018

Tomorrow we will be in 2019 and we still can't represent a current job.

@j-ecker
Copy link

j-ecker commented Jan 22, 2019

We are currently in 2019, has there been any resolution to this?

@nrgapple
Copy link

has anyone made a fork with present integrated?

@evanplaice
Copy link
Member

evanplaice commented Mar 27, 2020

A current|present value could be represented by an empty string. For that change to take place #348 will have to be approved and merged first.

@Velua
Copy link

Velua commented Aug 4, 2020

What happened with this?

@thomasdavis
Copy link
Member

Most themes have implemented that an endDate with the value null or simply missing means that you are still at the job.

@danguilherme
Copy link

Most themes have implemented that an endDate with the value null or simply missing means that you are still at the job.

Should the schema change? The latest one doesn't allow an empty string or null as value.

@thomasdavis
Copy link
Member

Yeah we should let it be set explicitly to null going fourth.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests