Skip to content

Just updated the new timetable excel File as communicated by college timetable has some changes. Also PG timetable is also added to the list. :) #59

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

Merged
merged 2 commits into from
Aug 3, 2024

Conversation

madhavkapila
Copy link
Contributor

No description provided.

…timetable has some changes. Also PG timetable is also added to the list. :)
@madhavkapila
Copy link
Contributor Author

The new timetable as communicated by authorities is updated. Also PG timetable is added to list. @utkarsh-1905 sir what a helpful, accessible website you created :) Thank you.

@madhavkapila
Copy link
Contributor Author

@utkarsh-1905 sir I am again getting restyler error. I locally tried increasing docker memory to 1024m also but don't want to commit until I know it will help or not

@utkarsh-1905
Copy link
Owner

Hey, ignore restyled errors, can you verify that time table is correct by randomly opening some classes, also does it contain the iep time table like for 2UoQ

@madhavkapila
Copy link
Contributor Author

Hey, ignore restyled errors, can you verify that time table is correct by randomly opening some classes, also does it contain the iep time table like for 2UoQ
Screenshot from 2024-08-03 19-53-55
Screenshot from 2024-08-03 19-55-04
Screenshot from 2024-08-03 19-55-48

Yes sir it containes IEP groups and also new PG sheet in it. I have attached screenshots for reference.

@utkarsh-1905
Copy link
Owner

lgtm

@utkarsh-1905 utkarsh-1905 merged commit 7ce92f6 into utkarsh-1905:main Aug 3, 2024
1 check failed
@madhavkapila
Copy link
Contributor Author

@utkarsh-1905 Sir now I viewed live timetable on website it does not have updates. Though the local one is still showing changes. Can you tell why it is happening? Please guide or it takes time to get timetable updated on live server which I think is not possible.

@utkarsh-1905
Copy link
Owner

Hey, it is deployed on my server, and I need to run a script to deploy the latest version, I'll do it tomorrow.

However, if a deployment pipeline was in place this can happen automatically, it would be great if someone creates that

@utkarsh-1905
Copy link
Owner

@madhavkapila deployed, raise a pr to add your name in the readme

@utkarsh-1905
Copy link
Owner

good work

@madhavkapila
Copy link
Contributor Author

good work

Thanks @utkarsh-1905 Sir your guidance during start of this in previous sem motivated me to contribute somewhat I know to the project. Though now completely focussing on DSA and CP, trying to learn dev also side by side but will dedicatedly do that after DSA is completed.

@madhavkapila
Copy link
Contributor Author

@madhavkapila deployed, raise a pr to add your name in the readme

If you not mind please allow me to do that during evening

@madhavkapila
Copy link
Contributor Author

@madhavkapila deployed, raise a pr to add your name in the readme

If you not mind please allow me to do that during evening

@utkarsh-1905 Sir updated Readme and created PR #60 .Thanks for such a motivating gesture :).

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.

2 participants