-
Notifications
You must be signed in to change notification settings - Fork 6
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
update schedule section of cookiecutter #41
Conversation
Just copying over the comment from @JessicaS11: @lsetiawan in my most recent commit I intentionally broke the cookiecutter/splash page by adding a few items to the
Alternatively, if you'd like to recommend an approach that is not me trying to brute-force, manually write and format html correctly I'd be willing to try and work on some of these. : ) |
@JessicaS11 I won't have time today to look over this. But I will check it out tomorrow! I think I understand what you're asking. Shouldn't be to difficult 👍🏽 |
21cc7b7
to
f8f6103
Compare
@JessicaS11 I got all your requested features I think. Let me know if I missed anything. Also please help on the build 😭 |
5287591
to
f30ecf1
Compare
Thanks @lsetiawan - this looks great (I am quite satisfied)! It builds fine for me locally, so let's see what GH has to say. |
Anyone know why the "this branch is out of sync with the base" update button isn't showing up? Sure I can merge manually, but that button is mighty handy... |
Not sure, but if you can merge the current main branch in that would be great! |
Far fetched guess is that there might be a merge conflict, which needs be resolved locally and hence no button here? UPDATE: As i typed this the button appeared. GitHub hiccup? |
Oops 🙊 hopefully that didn't mess anything up... I pressed the update branch button thing! Sorry! |
Perfect! I was lamenting the lack of update branch button! |
Jessica, after editing the |
🚀 Deployed on https://deploy-preview-41--hackweek-template.netlify.app |
As per this commen from #39