Update the list of global events

events/pull-meetup.js

@@ -12,31 +12,20 @@ const defaults = {
 }
 const results = []
 
-function clean (event) {
-  delete event.topics
-  delete event.urlname
-  delete event.category
-  delete event.id
-}
-
 function finish (events) {
-  // return console.log(JSON.stringify(events))
   events.forEach((event) => {
     if (!countryMap[event.country]) {
       console.log(event)
       throw new Error('Do not have map for ' + event.country)
     }
     const region = yml.getRegion(countryMap[event.country])
     if (!region.meetups) region.meetups = []
-    clean(event)
     if (!yml.isSoT(region.meetups, event.city, event.name)) {
       yml.replace(region.meetups, 'name', event.name, event)
     }
   })
   yml.save()
 }
-// This is nice when testing if you cache the response
-// finish(JSON.parse(require('fs').readFileSync('./meetup.json').toString()))
 
 function pull (opts) {
   request(opts, (err, resp, body) => {
@@ -47,9 +36,12 @@ function pull (opts) {
     body.results.forEach((result) => {
       const title = result.name.toLowerCase()
 
-      if (title.includes('nodeschool')) return
-      if (title.includes('mongodb') && title.includes('node')) return
-      if (title.includes('find a tech job') && title.includes('node')) return
+      if (
+        title.includes('find a tech job') ||
+        title.includes('nodeschool') ||
+        / mongodb (?:user group|meet ?up)$/.test(title) ||
+        title.startsWith('mongodb ')
+      ) return
 
       results.push(result)
     })
@@ -65,9 +57,9 @@ function pull (opts) {
 pull(Object.assign({
   url: 'https://api.meetup.com/2/groups',
   qs: {
+    only: 'city,country,description,group_photo.photo_link,lat,link,lon,name',
     key: process.env.MEETUP_TOKEN,
-    upcoming_events: true,
-    topic: 'nodejs',
-    category: 34
+    topic: 'nodeJS',
+    category_id: 34 // tech
   }
 }, defaults))

events/pull-nodeschool.js

@@ -29,20 +29,30 @@ request({
     }
 
     reg.chapters.forEach((chapter) => {
-      delete chapter.region
-      chapter.location = `${chapter.location}, ${chapter.country}`
+      if (chapter.country) {
+        chapter.location += `, ${chapter.country}`
+      }
       delete chapter.country
+      delete chapter.region
       yml.replace(store.nodeschools, 'name', chapter.name, chapter)
       chapters.push(chapter)
     })
   })
 
   function _geo () {
-    if (chapters.length === 0) { return yml.save() }
+    if (chapters.length === 0) {
+      return yml.save()
+    }
+
     const chapter = chapters.shift()
     geocoder.geocode(chapter.location, (err, res) => {
-      console.log(err, res)
-      if (err || !res.length) { return _geo() }
+      if (err || !res.length) {
+        console.error(
+          err && err.message || `Could not geocode location: ${chapter.location}`
+        )
+        return _geo()
+      }
+
       chapter.lat = res[0].latitude
       chapter.lon = res[0].longitude
       _geo()

events/yaml-sync.js

@@ -5,31 +5,35 @@ const fs = require('fs')
 const path = require('path')
 
 const p = path.join(__dirname, '..', 'locale', 'en', 'get-involved', 'events.md')
-const lines = fs.readFileSync(p).toString().split('\n')
-const begin = lines.indexOf('---') + 1
-const end = lines.indexOf('---', begin)
-const store = yaml.safeLoad(lines.slice(begin, end).join('\n'))
-
-function getRegion (region) {
-  let reg
-  for (reg in store.regions) {
-    if (store.regions[reg].region === region) return store.regions[reg]
+
+//
+// Slice the file contents to get the YAML source code.
+//
+const contents = fs.readFileSync(p, { encoding: 'utf8' }).trim().slice(4, -4)
+const store = yaml.safeLoad(contents)
+
+store.regions || (store.regions = [])
+
+function getRegion (name) {
+  let region = store.regions.find((reg) => reg.region === name)
+
+  if (!region) {
+    region = { region: name }
+    store.regions.push(region)
   }
-  reg = { region: region }
-  store.regions.push(reg)
-  return reg
+
+  return region
 }
 
+/**
+ * This function checks if an event has been manually edited to prevent it
+ * from being overwritten the next time event scripts are run.
+ *
+ * See https://github.com/nodejs/nodejs.org/pull/398.
+ */
 function isSoT (meetups, city, name) {
-  for (let i = 0; i < meetups.length; i++) {
-    if (meetups[i].city === city && meetups[i].name === name) {
-      if (meetups[i].source_of_truth) {
-        return true
-      }
-      return false
-    }
-  }
-  return false
+  const meetup = meetups.find((evt) => evt.city === city && evt.name === name)
+  return meetup && meetup.source_of_truth
 }
 
 function removeEmpty (dict) {
@@ -39,19 +43,23 @@ function removeEmpty (dict) {
 }
 
 function replace (list, key, keyValue, value) {
+  const index = list.findIndex((elem) => elem[key] === keyValue)
+
   removeEmpty(value)
-  for (let i = 0; i < list.length; i++) {
-    if (list[i][key] === keyValue) {
-      list[i] = value
-      return
-    }
+
+  if (index !== -1) {
+    list[index] = value
+  } else {
+    list.push(value)
   }
-  list.push(value)
 }
 
 function save () {
-  const str = ['---', yaml.dump(store), '---'].join('\n')
-  fs.writeFileSync(p, str)
+  fs.writeFileSync(p, [
+    '---',
+    yaml.safeDump(store, { lineWidth: Infinity }),
+    '---'
+  ].join('\n'))
 }
 
 exports.removeEmpty = removeEmpty