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

Load yaml file data #16

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Load yaml file data #16

wants to merge 15 commits into from

Conversation

xwang345
Copy link

No description provided.

@xwang345
Copy link
Author

xwang345 commented Nov 15, 2018

I am still working on...
test

@ywpark1
Copy link
Owner

ywpark1 commented Nov 16, 2018

Thanks.

Can I ask why you removed portfolio directory from .gitignore file? I was putting it because all files in that directory are generated by the user for their portfolio, and we have the same files in templates directory.

Also, can you provide more information about this PR? We want to know more about this.

@xwang345
Copy link
Author

xwang345 commented Nov 16, 2018

Yeah, I did remove the .gitignore because I want to modify your original HTML template file. At the same time, I want Git track what I doing in the template folder. I add a {} to receive the data that come from the cs file.

html

You can add any personal information in the yaml file

code

this is how it look like when type dotnet run --project ./Portfolio-generator-console/Portfolio-generator-console.csproj

web

Its display the yaml file informations.

@xwang345
Copy link
Author

xwang345 commented Nov 16, 2018

I think this web app user don't want go to html to insert their information, so I use yaml file to store their info. then c# parse the yaml file and render it to the HTML.

@ywpark1
Copy link
Owner

ywpark1 commented Nov 16, 2018

What I was thinking was generating html file from user's resume.

User uploads the resume, and this program extracts the data from the resume file(.docx), and put the data into the template by replacing the placeholder.

I did not want users to enter all information manually.

@ywpark1
Copy link
Owner

ywpark1 commented Nov 16, 2018

That is why we were trying to work on #11 . I think your idea is not bad if we can convert docx to your yml file.

@xwang345
Copy link
Author

Okay I understand. I can work on this feature.

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.

None yet

2 participants