-
Notifications
You must be signed in to change notification settings - Fork 24
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
stub out new nyc event at Reaktor #125
stub out new nyc event at Reaktor #125
Conversation
Could you make a new parallel yaml file rather than modifying the existing one? See Berlin for an example of a city that has hosted multiple events. |
o sure; that actually makes a lot more sense, thanks for the tip! |
hmm, the CI jobs are failing now, and i don't know why? maybe the |
@edwardsharp @littledan Build fixed and now we have Github actions so if it fails everyone can see why |
Huh, it doesn't seem to show us organizers on the page. Are we supposed to be mainOrganizer, or mc, to show up? |
amazing and thank you @SaraVieira 🥳 one thing, is maybe (to double-check), did you mean to commit the |
@littledan i moved it back to |
@edwardsharp @littledan Okay that was a a weird code choice I made there, fixed it. Sorry I didn't have much time to update the website so it was all super wonky also yeah i put the env by choice, airtable now has role based API keys so this one is just a read |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ship it!
@@ -70,9 +72,10 @@ const Main = ({ city, attendees }) => { | |||
<Sponsors sponsors={sponsors} /> | |||
</Panel> | |||
)} | |||
{mainOrganizer && mainOrganizer.length > 0 ? ( | |||
<Panel heading={mainOrganizer.length > 1 ? 'Organizers' : 'Organizer'}> | |||
<Organizers organizers={mainOrganizer} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change will affect the rendering of some existing yaml files, but maybe in a good way, e.g.,
website/data/online-2020-16.yaml
Line 19 in dc438b3
mainOrganizer: |
@littledan and @SaraVieira i pushed one thing where the organizers anyhow, yeah, feel like this could merge?? |
Sounds good! Ship it!! |
okay! i've shipped it as far as i can! i don't have the permissions to merge this, so i need some help from someone else 🥺 |
Done!! |
initial details for nyc meetup at Reaktor office thursday, april 25, 2024 from 6-10pm.
hoping to drum up some proposals so will be adding more speaker details soon!
i don't have the env vars needed to run this successfully on my localhost machine;
so i hope this looks fine.edit: ooo, but there's a preview deployment! https://deploy-preview-125--queerjs.netlify.app/new-york/ 😎
date: 04/25/2024
assumed month/day/year format. (looking at the preview build now, i think that's correct ✅)