Closed nikhilch23 closed 5 years ago
Great page <3 A few suggestions:
cc/ @dibyadas @djbarnwal @nikhilch23
Awesome! @thealphadollar Can you make the server up and running with this as the landing page?
@dibyadas I would wait for the changes to be made :) I also have to see if the form is a dummy one or actually working.
Definitely. That's what I meant. :+1:
Should i only replace the previous app.py with mine? or remove all the folders all together?
cc @thealphadollar @dibyadas
No; do not remove anything! Keep your folder separate, just follow the pattern of kwoc/app.py for your app.py.
Please do not remove anything outside your folder.
@thealphadollar I have made the necessary changes. Do check.
@nikhilch23 Please note that you should not send PRs from the master branch
@nikhilch23
@dibyadas I've a lab test tomorrow, I'll deploy tomorrow only. Once all the work is done :)
I have a test tomorrow morning. If someone is free, could you help me with the last issue that dhiraj has suggested? @xypnox @yashrsharma44 _
@dibyadas What should be the time remaining?
@thealphadollar Let's follow last years schedule.
@sangeet259 Let's move it by 3 days forward for first two items of the schedule? We don't have a finalized coming soon page yet.
@thealphadollar Yeah, that's better, shift it a bit.
Hey, if the work is not done, I am willing to complete it @thealphadollar
Please do @yashrsharma44. I would like you to work on a separate branch and make changes (after I've merged the changes done here).
I'll create an issue listing what all needs to be done before we go live.
@kshitij10496 @djbarnwal I'm merging the PR and then @yashrsharma44 will be working further on it. I'm doing this since we need to speed up things; I hope you don't mind.
Hey! i guess the problem is reolved now. Do check. @thealphadollar