Merge branch 'release/1.9.0' into prod

README.md

@@ -153,3 +153,27 @@ On utilise également cet appel dans la page de status, pour vérifier que la co
 
 - Obtenir l'historique d'une conf passée donnée : `GET /telephony/${process.env.OVH_ACCOUNT_NUMBER}/conference/${phoneNumber}/histories/${callId}`
 
+## Mise en prod
+La mise en prod est faite à la main pour le moment.
+
+Procédé basique :
+ - On fait une branche release:
+ ```
+git checkout main
+git pull
+git checkout -B release/X.Y.Z
+ ```
+ - On incrémente le numéro de version dans package.json et on commit dans la branche de release. On teste cette branche de release.
+ - on merge la release dans la branche `prod`
+ ```
+ git checkout prod
+ git merge --no-ff release/X.Y.Z
+ ```
+ - on fait une release sur github (https://github.com/betagouv/audioconf/releases/new) en décrivant les modifications apportées par cette release. Ce process crée un tag sur le commit de la release.
+ - on déploie la branche prod sur Scalingo.
+ - on merge `prod` dans `main` (pour récupérer le numéro de version et les éventuelles modifs de test):
+ ```
+ git checkout main
+ git pull
+ git merge --no-ff prod
+ ```

controllers/createConfController.js

@@ -31,7 +31,7 @@ const createConfWithDay = async (email, conferenceDay, userTimezoneOffset) => {
 
     const OVHconfData = await conferences.createConf(freeAt)
 
-    const conference = await db.insertConferenceWithFreeAt(email, OVHconfData.phoneNumber, OVHconfData.freeAt)
+    const conference = await db.insertConferenceWithDay(email, OVHconfData.phoneNumber, conferenceDay, OVHconfData.freeAt)
     conference.pin = OVHconfData.pin
     return conference
   } catch (err) {
@@ -101,12 +101,6 @@ module.exports.showConf = async (req, res) => {
       return res.redirect('/')
     }
 
-    // Whether this conf was booked for the whole day.
-    conference.isDayConference = false
-    if (!conference.durationInMinutes) {
-      conference.isDayConference = true
-    }
-
     res.render('confCreated', {
       pageTitle: 'Votre conférence',
       conference

index.js

@@ -20,6 +20,8 @@ if (config.NODE_ENV === "development") {
   console.dir({ config })
 }
 
+const version = require("./package.json").version
+
 const app = express()
 
 app.set("view engine", "ejs")
@@ -57,6 +59,7 @@ app.use(function(req, res, next){
   res.locals.infos = req.flash("info")
   res.locals.successes = req.flash("success")
   res.locals.urls = urls
+  res.locals.version = version
   next()
 })
 

lib/db.js

@@ -37,7 +37,7 @@ module.exports.insertConference = async (email, phoneNumber, durationInMinutes,
     }
 }
 
-module.exports.insertConferenceWithFreeAt = async (email, phoneNumber, expiresAt) => {
+module.exports.insertConferenceWithDay = async (email, phoneNumber, conferenceDay, expiresAt) => {
   const domain = parseEmail(email).domain
   const hashedEmail = format.hashForLogs(email)
 
@@ -49,6 +49,7 @@ module.exports.insertConferenceWithFreeAt = async (email, phoneNumber, expiresAt
         expiresAt,
         domain,
         hashedEmail,
+        conferenceDay,
       })
       .returning("*")
 
@@ -64,6 +65,13 @@ module.exports.getUnexpiredConference = async (id) => {
         const conferences = await knex("conferences").select()
             .where({ id: id })
             .andWhere("expiresAt", ">", new Date())
+        // Format the conferenceDays to be YYYY-MM-DD strings, rather than full Date objects
+        // (the field is a table.date in knex, not a table.datetime, so it's less confusing to use a dateString.)
+        conferences.forEach(conference => {
+          if (conference.conferenceDay) {
+              conference.conferenceDay = format.formatStandardDate(conference.conferenceDay)
+          }
+        })
         return conferences[0]
     } catch (err) {
       console.error("getUnexpiredConference: Impossible de récupérer la conférence %s", id, err)

migrations/20210122164657_add-conference-day-in-conferences.js

@@ -0,0 +1,11 @@
+exports.up = function (knex) {
+  return knex.schema.table("conferences", function (table) {
+    table.date("conferenceDay")
+  })
+}
+
+exports.down = function (knex) {
+  return knex.schema.table("conferences", function (table) {
+    table.dropColumn("conferenceDay")
+  })
+}

package.json

@@ -1,6 +1,6 @@
 {
   "name": "conferences",
-  "version": "1.8.0",
+  "version": "1.9.0",
   "description": "Conférences audio et vidéo pour les fonctionnaires de l'Etat",
   "main": "index.js",
   "directories": {