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

Update to match the latest Resume.JSON format #66

Merged
merged 2 commits into from
Aug 22, 2024

Conversation

housamz
Copy link
Contributor

@housamz housamz commented Aug 9, 2024

Changes

  • Added the following keys in the Output object: volunteer, awards, certificates.
  • Updated Position to have keys name and url instead of company and website.
  • Updated Education to have keys score instead of gpa and added url.
  • Fixed the empty fields in basics.location by splitting geoLocation into city and country.
  • Updated the CountryCode search to use the Country key in the geoLocation object.
  • Added Regex to extract the website and insert it in basics.url.
  • Updated basics.picture to be basics.image.
  • Updated basics.website to be basics.url.
  • updated project.summary to be project.description.
  • Added endDate and highlights in project.
  • Updated publications.website to be publications.url.
  • Added the following feature:
    • processAwards: Extracts the awards from the Honors.csv file and adds them to the Output object.
    • processCertificates: Extracts the certificates from the Certifications.csv file and adds them to the Output object. The certificates are sorted by date.
    • processEndorsements: Extracts the endorsements from the Endorsement_Received_Info.csv file and use them to insert a level in skills. The skills are sorted by level.
    • processFollows: Extracts the followers from the Causes You Care About.csv, Company Follows.csv, and Hashtag_Follows.csv files and use them to fill the interests.

@thomasdavis
Copy link

Really good work @housamz

There have been quite a few issues because of the schema changes (my fault for not having a good pipeline for it)

@housamz
Copy link
Contributor Author

housamz commented Aug 9, 2024

@thomasdavis Thanks very much for your comment (and the JSON resume).

@JMPerez
Copy link
Owner

JMPerez commented Aug 21, 2024

Thank you so much for your work, @housamz!

Could you check the error preventing the build from working? I believe you can just drop the ? in this.target.skills? as it is assumed to be an array just before, when calling .map.

@housamz
Copy link
Contributor Author

housamz commented Aug 22, 2024

Done, thanks

@JMPerez JMPerez merged commit 5d8b017 into JMPerez:master Aug 22, 2024
1 check passed
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.

3 participants