-
Notifications
You must be signed in to change notification settings - Fork 90
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
Created an upcoming events widget #192
base: master
Are you sure you want to change the base?
Conversation
Looking awesome. We'll need to figure out the workflow for editing the event should it change. It will sorta suck tracking down the admin to make an edit to time/location/etc. Is the spreadheet going to be shared with all admins or controlled by just a few? |
I'd say any organizer should have access. We can also set the option on the form to have it email the responder a link to update their event if they need to. |
bump. There's a setting on the Google form that will send an email to whoever completes the form with a link to update their response. Will that suffice from an admin/management perspective? |
I'm a 👍 |
After an obscene amount of time figuring out why I couldn't preview the site locally I was finally able to check this out. Looks great to me. My only concern would be I'm anticipating difficulty getting organizers to routinely fill out the form. Eventually, it would be nice to automate this for those chapters using a platform that has an API (e.g. eventbrite, meetup). But this works well for the time being. Thoughts? @jonahadkins @maptime/owners |
Looking good, @geobrando! Problems getting Ruby dependencies lined up I'm guessing? I totally wasn't up all night doing the same thing last night... Anywho, for Maptime Chicago's events, I'm using the YAML header date variable that (I think?) @geografa set up in the starter template. I bet we could pull that info into the widget, though it might be more work than it's worth.
The event info is mostly here and the date comes from the filename as is Jekyll's wont. |
Using the meetup api would be nice but using meetup is optional due to cost. Would be something cool to target for funding by hq when the time comes. Roy
|
Gentle nudge on behalf of @kkowalsky to get this plan and site change reviewed by some HQ folks so that we can start implementing it. We'd like to include an event calendar in the monthly series in xyHT. Roy set this up. But basically, organizers will be asked to fill out a google form when they plan events and this will populate a google sheet and the events calendar on the site as shown above. |
@mapsam Should we go ahead and modify the google form to get it ready for production? It currently says "TEST": https://docs.google.com/forms/d/1OfA8sV0Vs0-WlCwRBALC8p_RR1zMFS-WXYtXnyDzTA0/viewform |
Oh, hehe. Feel free to make a new form if you'd like - otherwise let me know what you'd like the form to say. |
@mapsam I just can't seem to find it in my Google Drive. I know you shared it with me... |
Here's the link to edit |
Hardcoded chapter names will be a pain to maintain 😢 |
@rhewitt22 Hmm.. you could keep it sync'd chapters.json. But not sure of any of us have the bandwidth to figure that out. |
Google App Script HTTP request to raw git? https://cdn.rawgit.com/maptime/maptime.github.io/master/_data/chapters.json Not sure if it's even possible to update a choice list programmatically. |
Remember you can change that form to whatever you want - it was just created as a proof of concept a long time ago! Might as well start with a text field. |
Yea I'm just thinking how this might be used on Chapter pages to limit events to only those hosted by their chapter. If this is a text field that adds some degree of user error with capitalization and spaces. I guess that could all be handled by JS on the client (toLowerCase(), trim()). |
I assumed this was only intended as a worldwide aggregator for maptime.io. Not sure how desired this would be for chapters as if they're using the starter template it already highlights the next event and will link to a meetup site. |
I don't see any reason why modularity is a bad thing. If a chapter wants to implement this in order to show more than one upcoming event why shouldn't they be able to? It's not a requirement, it's an additional option. |
Oh I agree Roy. I'm all for that. Was just saying if it's going to create additional headaches it might be best to keep it simple. But yeah, you could probably figure out how to handle some fuzzy matching. |
It's a balance. I'm guessing a little fuzzy matching would be the easiest/most maintainable option compared to hard-coded chapters, or spending the time to figure out if you can programmatically update a pick-list on a Google form. Will look into this soon. |
Addresses #105.
Updated
site.js
to query a google spreadsheet using tabletop.js for maptime events. The script ditches events that have already occurred, and sorts them by date before rendering them in a list in the sidebar.Events list is hidden by default on mobile, they are shown when you click the Upcoming events header. Shown by default in a scrollable list in the sidebar on larger screens.
Caveat: since last november when the google form was created Google changed they way they save dates to the spreadsheet. They used to save in an ISO format that moment.js liked. Now dates are saved in the following format 6/8/2015 18:30:00, which causes moment.js to throw a warning: