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

Loan Calculator Finance Tool #65

Open
1 of 6 tasks
Rearo43 opened this issue Oct 8, 2020 · 4 comments
Open
1 of 6 tasks

Loan Calculator Finance Tool #65

Rearo43 opened this issue Oct 8, 2020 · 4 comments

Comments

@Rearo43
Copy link

Rearo43 commented Oct 8, 2020

Create a Loan Calculator in html,css and javascript.


  • Add navbar in your tool's html file. You can copy the navbar from index.html.
  • Add screenshot of your tool in, PR message and also in the folder of your tool
  • Before Making the PR Pull the latest changes that have been made to the repository

**Note: Please Don't Make Any Changes In `index.html` AND `contributors.html` **


Add the below mention section to your PR message


Title


Number of files/folder added


Changes made


Check the options that you have completed,


I have;

  • Read contibuting.md
  • Followed all the guidelines
  • updated readme.md
  • add your name in the footer of your tool.html file (if you created length conversion tool then add your name to length.html file you created)
  • provide below mentioned details if you want your name to be added in contributors.html
  • mail, twitter, github profile link.
@Rearo43
Copy link
Author

Rearo43 commented Oct 8, 2020

May I please be assigned to the Loan Calculator in Finance Tools?

@Rearo43
Copy link
Author

Rearo43 commented Oct 10, 2020

Would you be willing to use EJS templating instead of copying all the nav bar code and other features from the index.js file? Contributors would be able to just write one line, "<%- include('../../layout/nav') %>" for example into their code. That one line would import all the navbar code to the contributors file and eliminate any risk of typos, keeping the whole site layout completely uniform. If you were into using it I would be more than happy to go through and set up the file structure and update the README with instructions. This would also make the workflow much faster and simpler.

@shubham2704
Copy link
Member

Yes it would be great and

Would you be willing to use EJS templating instead of copying all the nav bar code and other features from the index.js file? Contributors would be able to just write one line, "<%- include('../../layout/nav') %>" for example into their code. That one line would import all the navbar code to the contributors file and eliminate any risk of typos, keeping the whole site layout completely uniform. If you were into using it I would be more than happy to go through and set up the file structure and update the README with instructions. This would also make the workflow much faster and simpler.

Yes it would be a great feature. Our team is planning on that, we will be adding the functionality of handlebars and middlewares . we will change the contributing guidelines as soon as this happens. But until then the guidelines remains the same and all the contributors have to manage with copy/paste, Because we have to also account for the beginner developers/contributors.

@shubham2704
Copy link
Member

@Rearo43 Are you still working on this?

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

No branches or pull requests

2 participants