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 VIT AP campus #124

Closed
ashua4488 opened this issue Jul 1, 2019 · 7 comments · Fixed by #247
Closed

Add support for VIT AP campus #124

ashua4488 opened this issue Jul 1, 2019 · 7 comments · Fixed by #247
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers priority/low This issue doesn't have to be resolved immediately

Comments

@ashua4488
Copy link

HOW ABOUT CREATING SIMILAR FUNCTIONALITY FOR FFCS AP CAMPUS. THOSE GUYS ARE REALLY STRUGGLING.

@AnmolTomer
Copy link
Contributor

It just needs a similar excel file as we have for Vellore and Chennai campus. Check it out inside the util directory in the master branch of the repo. They have to make that and that would be a good start.

@ashua4488
Copy link
Author

ashua4488 commented Jul 2, 2019 via email

@therealsujitk therealsujitk added enhancement New feature or request priority/low This issue doesn't have to be resolved immediately labels Dec 11, 2021
@therealsujitk therealsujitk changed the title FFCS for VIT AP CAMPUS Add support for VIT AP campus Dec 28, 2021
@therealsujitk therealsujitk added the good first issue Good for newcomers label Dec 31, 2021
@anantgupta001
Copy link
Contributor

anantgupta001 commented Jun 26, 2024

Hello @AnmolTomer,

I have written the code for the VIT-AP campus in my local repository. However, we do not receive faculty names in our annexure PDF during course registration, as seen in the example PDF I have uploaded. Could you please advise on how to obtain the faculty names, or should I list them as Faculty-1, Faculty-2, Faculty-3 sequentially?

Thank you.

ANNEXURE-II(A) Theory Slots.pdf
ANNEXURE-II(B) Lab Slots.pdf

@therealsujitk
Copy link
Collaborator

@anantgupta001, for now we can try testing it leaving the field blank. You'll need to make a single .xlsx file with the required columns (mentioned here --> https://github.com/vatz88/FFCSonTheGo/blob/master/util/README.md). The values for the columns that you do not have (like faculty) can be left blank.

@anantgupta001
Copy link
Contributor

Hi @therealsujitk,

I am currently working on the project and would like to formally request to be assigned this task. I believe my skills and experience align well with the requirements, and I am confident in my ability to contribute effectively.

Thank you for considering my request.

Best regards,
Anant Gupta

@therealsujitk
Copy link
Collaborator

Sure @anantgupta001! Thankyou for contributing, let me know if you need any help.

@anantgupta001
Copy link
Contributor

Hi @therealsujitk,

I have completed the edits to the course panel and attached the 'report_ap' file containing the relevant data. The changes have passed all necessary tests and can be used for testing purposes.

Could you please review and merge the changes at your earliest convenience?

Thank you for your attention to this matter.

Best regards,
Anant Gupta

@therealsujitk therealsujitk linked a pull request Jun 28, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers priority/low This issue doesn't have to be resolved immediately
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants