seatsio 69.1.0 → 71.0.0

package/.github/workflows/build.yml

@@ -15,11 +15,11 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
-          node-version: 13
+          node-version: 18
           registry-url: https://registry.npmjs.org/
-      - run: npm install
-      - run: npm run lint
+      - run: yarn install
+      - run: yarn lint
       - uses: zcong1993/setup-timezone@master
         with:
           timezone: Europe/Brussels
-      - run: npm run test
+      - run: yarn test

package/.github/workflows/publish.yml

@@ -11,16 +11,9 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
-          node-version: 13
+          node-version: 18
           registry-url: https://registry.npmjs.org/
-      - run: npm install
-      - run: npm run bundle
-      - run: npm publish
+      - run: yarn install
+      - run: yarn publish
         env:
           NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}
-      - uses: svenstaro/upload-release-action@v2
-        with:
-          repo_token: ${{ secrets.GITHUB_TOKEN }}
-          file: build/SeatsioClient.js
-          tag: ${{ github.ref }}
-          overwrite: true

package/README.md

@@ -20,23 +20,6 @@ For Node, you can install using npm:
 ```sh
 npm install seatsio
 ```
-Then you can use it with require: 
-```js
-const { SeatsioClient } = require('seatsio')
-```
-For browser, you can directly include it from GitHub:
-
-```html
-<script src="https://github.com/seatsio/seatsio-js/releases/download/<RELEASE_TAG>/SeatsioClient.js"></script>
-```
-
-This library uses `async/await` introduced with ES2017 and `for await` loops which is part of the ES2018. `for await` loops gained native support from Node 10 onwards. As a result, the minimum required version for Node is 10. 
-
-## Versioning
-
-seatsio-js follows semver since v54.4.0.
-
-Please note that any version below v2 is not production ready.
 
 ## Usage
 

package/package.json

@@ -1,13 +1,12 @@
 {
   "name": "seatsio",
-  "version": "69.1.0",
+  "version": "71.0.0",
   "main": "index.js",
   "description": "Official JavaScript and Node.JS client library for the Seats.io REST API",
   "license": "MIT",
   "scripts": {
-    "lint": "eslint ./src",
-    "test": "jest --maxWorkers=8 --verbose",
-    "bundle": "gulp bundle"
+    "lint": "eslint ./src ./tests",
+    "test": "jest --maxWorkers=8 --verbose"
   },
   "repository": {
     "type": "git",
@@ -17,21 +16,13 @@
     "axios": "0.27.2"
   },
   "devDependencies": {
-    "browserify": "latest",
-    "eslint": "7.32.0",
-    "eslint-config-standard": "16.0.3",
+    "eslint": "8.26.0",
+    "eslint-config-standard": "17.0.0",
     "eslint-plugin-import": "2.26.0",
-    "eslint-plugin-node": "11.1.0",
-    "eslint-plugin-promise": "6.0.0",
-    "eslint-plugin-standard": "5.0.0",
-    "gulp": "latest",
-    "gulp-buffer": "latest",
-    "gulp-jest": "latest",
-    "gulp-tap": "latest",
-    "gulp-uglify": "latest",
-    "gulp-uglify-es": "latest",
-    "jest": "28.1.0",
-    "jest-cli": "28.1.0",
-    "uuid": "8.3.2"
+    "eslint-plugin-n": "15.5.0",
+    "eslint-plugin-promise": "6.1.1",
+    "jest": "29.2.2",
+    "jest-cli": "29.2.2",
+    "uuid": "9.0.0"
   }
 }

package/src/Common/IDs.js

@@ -7,5 +7,5 @@ class IDs {
 }
 
 module.exports = {
-    IDs: IDs
+    IDs
 }

package/src/Common/Labels.js

@@ -21,6 +21,6 @@ class LabelAndType {
 }
 
 module.exports = {
-    Labels: Labels,
-    LabelAndType: LabelAndType
+    Labels,
+    LabelAndType
 }

package/src/Events/Channels.js

@@ -9,10 +9,10 @@ class Channels {
     add (eventKey, channelKey, name, color, index, objects) {
         return this.client.post(`/events/${encodeURIComponent(eventKey)}/channels`, {
             key: channelKey,
-            name: name,
-            color: color,
-            index: index,
-            objects: objects
+            name,
+            color,
+            index,
+            objects
         })
     }
 
@@ -30,22 +30,22 @@ class Channels {
 
     addObjects (eventKey, channelKey, objects) {
         return this.client.post(`/events/${encodeURIComponent(eventKey)}/channels/${encodeURIComponent(channelKey)}/objects`, {
-            objects: objects
+            objects
         })
     }
 
     removeObjects (eventKey, channelKey, objects) {
         return this.client.delete(`/events/${encodeURIComponent(eventKey)}/channels/${encodeURIComponent(channelKey)}/objects`, {
-            data: { objects: objects }
+            data: { objects }
         })
     }
 
     replace (eventKey, channels) {
-        return this.client.post(`/events/${encodeURIComponent(eventKey)}/channels/update`, { channels: channels })
+        return this.client.post(`/events/${encodeURIComponent(eventKey)}/channels/update`, { channels })
     }
 
     setObjects (eventKey, channelConfig) {
-        return this.client.post(`/events/${encodeURIComponent(eventKey)}/channels/assign-objects`, { channelConfig: channelConfig })
+        return this.client.post(`/events/${encodeURIComponent(eventKey)}/channels/assign-objects`, { channelConfig })
     }
 }
 

package/src/Events/Event.js

@@ -11,7 +11,7 @@ class Event {
         this.key = json.key
         this.tableBookingConfig = json.tableBookingConfig
         this.supportsBestAvailable = json.supportsBestAvailable
-        this.forSaleConfig = json.forSaleConfig ? new ForSaleConfig(json.forSaleConfig.forSale, json.forSaleConfig.objects, json.forSaleConfig.categories) : null
+        this.forSaleConfig = json.forSaleConfig ? new ForSaleConfig(json.forSaleConfig.forSale, json.forSaleConfig.objects, json.forSaleConfig.areaPlaces, json.forSaleConfig.categories) : null
         this.chartKey = json.chartKey
         this.createdOn = json.createdOn ? new Date(json.createdOn) : null
         this.updatedOn = json.updatedOn ? new Date(json.updatedOn) : null

package/src/Events/Events.js

@@ -169,34 +169,41 @@ class Events {
 
     /**
      * @param {string} eventKey
-     * @param {?object} objects
+     * @param {?string[]} objects
+     * @param {?object} areaPlaces
      * @param {?string[]} categories
      * @returns {Promise}
      */
-    markAsForSale (eventKey, objects = null, categories = null) {
+    markAsForSale (eventKey, objects = null, areaPlaces = null, categories = null) {
         const requestParameters = {}
         if (objects !== null) {
             requestParameters.objects = objects
         }
+        if (areaPlaces !== null) {
+            requestParameters.areaPlaces = areaPlaces
+        }
         if (categories !== null) {
             requestParameters.categories = categories
         }
+
         return this.client.post(`events/${encodeURIComponent(eventKey)}/actions/mark-as-for-sale`, requestParameters)
     }
 
     /**
      * @param {string} eventKey
-     * @param {?object} objects
+     * @param {?string[]} objects
+     * @param {?object} areaPlaces
      * @param {?string[]} categories
      * @returns {Promise}
      */
-    markAsNotForSale (eventKey, objects = null, categories = null) {
+    markAsNotForSale (eventKey, objects = null, areaPlaces = null, categories = null) {
         const requestParameters = {}
-
         if (objects !== null) {
             requestParameters.objects = objects
         }
-
+        if (areaPlaces !== null) {
+            requestParameters.areaPlaces = areaPlaces
+        }
         if (categories !== null) {
             requestParameters.categories = categories
         }

package/src/Events/ForSaleConfig.js

@@ -2,10 +2,12 @@ class ForSaleConfig {
     /**
      * @param {boolean} forSale
      * @param {string[]} objects
+     * @param {object} areaPlaces
      * @param {string[]} categories
      */
-    constructor (forSale, objects, categories) {
+    constructor (forSale, objects, areaPlaces, categories) {
         this.forSale = forSale
+        this.areaPlaces = areaPlaces
         this.objects = objects
         this.categories = categories
     }

package/src/Events/StatusChangesParams.js

@@ -74,7 +74,7 @@ class StatusChangesParams {
         return {
             filter: this.filter,
             filterType: this.filterType,
-            sort: sort
+            sort
         }
     }
 }

package/src/Reports/ChartReports.js

@@ -26,7 +26,7 @@ class ChartReports {
     /**
      * @param {string} chartKey
      * @param {string} bookWholeTables
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryByObjectType (chartKey, bookWholeTables = undefined) {
         return this.fetchSummaryReport('byObjectType', chartKey, bookWholeTables)
@@ -44,7 +44,7 @@ class ChartReports {
     /**
      * @param {string} chartKey
      * @param {string} bookWholeTables
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryByCategoryLabel (chartKey, bookWholeTables = undefined) {
         return this.fetchSummaryReport('byCategoryLabel', chartKey, bookWholeTables)
@@ -62,7 +62,7 @@ class ChartReports {
     /**
      * @param {string} chartKey
      * @param {string} bookWholeTables
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryByCategoryKey (chartKey, bookWholeTables = undefined) {
         return this.fetchSummaryReport('byCategoryKey', chartKey, bookWholeTables)
@@ -80,7 +80,7 @@ class ChartReports {
     /**
      * @param {string} chartKey
      * @param {string} bookWholeTables
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryBySection (chartKey, bookWholeTables = undefined) {
         return this.fetchSummaryReport('bySection', chartKey, bookWholeTables)

package/src/Reports/EventReports.js

@@ -17,7 +17,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryByStatus (eventKey) {
         return this.client.get(EventReports.summaryReportUrl('byStatus', eventKey))
@@ -26,7 +26,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     deepSummaryByStatus (eventKey) {
         return this.client.get(EventReports.deepSummaryReportUrl('byStatus', eventKey))
@@ -45,7 +45,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryByObjectType (eventKey) {
         return this.client.get(EventReports.summaryReportUrl('byObjectType', eventKey))
@@ -54,7 +54,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     deepSummaryByObjectType (eventKey) {
         return this.client.get(EventReports.deepSummaryReportUrl('byObjectType', eventKey))
@@ -73,7 +73,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryByCategoryLabel (eventKey) {
         return this.client.get(EventReports.summaryReportUrl('byCategoryLabel', eventKey))
@@ -82,7 +82,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     deepSummaryByCategoryLabel (eventKey) {
         return this.client.get(EventReports.deepSummaryReportUrl('byCategoryLabel', eventKey))
@@ -101,7 +101,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryByCategoryKey (eventKey) {
         return this.client.get(EventReports.summaryReportUrl('byCategoryKey', eventKey))
@@ -110,7 +110,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     deepSummaryByCategoryKey (eventKey) {
         return this.client.get(EventReports.deepSummaryReportUrl('byCategoryKey', eventKey))
@@ -149,7 +149,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryBySection (eventKey) {
         return this.client.get(EventReports.summaryReportUrl('bySection', eventKey))
@@ -158,7 +158,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     deepSummaryBySection (eventKey) {
         return this.client.get(EventReports.deepSummaryReportUrl('bySection', eventKey))
@@ -187,7 +187,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryByAvailability (eventKey) {
         return this.client.get(EventReports.summaryReportUrl('byAvailability', eventKey))
@@ -196,7 +196,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryByAvailabilityReason (eventKey) {
         return this.client.get(EventReports.summaryReportUrl('byAvailabilityReason', eventKey))
@@ -205,7 +205,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     deepSummaryByAvailability (eventKey) {
         return this.client.get(EventReports.deepSummaryReportUrl('byAvailability', eventKey))
@@ -214,7 +214,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     deepSummaryByAvailabilityReason (eventKey) {
         return this.client.get(EventReports.deepSummaryReportUrl('byAvailabilityReason', eventKey))
@@ -233,7 +233,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     summaryByChannel (eventKey) {
         return this.client.get(EventReports.summaryReportUrl('byChannel', eventKey))
@@ -242,7 +242,7 @@ class EventReports {
 
     /**
      * @param {string} eventKey
-     * @returns {Object} JSON response from the server
+     * @returns {object} JSON response from the server
      */
     deepSummaryByChannel (eventKey) {
         return this.client.get(EventReports.deepSummaryReportUrl('byChannel', eventKey))

package/src/Subaccounts/Subaccounts.js

@@ -106,7 +106,7 @@ class Subaccounts {
      * @returns {AsyncIterator}
      */
     listAll (filter = null) {
-        const requestParameters = filter !== null ? { filter: filter } : {}
+        const requestParameters = filter !== null ? { filter } : {}
         return this.iterator().all(requestParameters)
     }
 
@@ -114,7 +114,7 @@ class Subaccounts {
      * @returns {Page}
      */
     listFirstPage (filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.iterator().firstPage(requestParameters, pageSize)
     }
 
@@ -122,7 +122,7 @@ class Subaccounts {
      * @returns {Page}
      */
     listPageAfter (afterId, filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.iterator().pageAfter(afterId, requestParameters, pageSize)
     }
 
@@ -130,7 +130,7 @@ class Subaccounts {
      * @returns {Page}
      */
     listPageBefore (beforeId, filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.iterator().pageBefore(beforeId, requestParameters, pageSize)
     }
 

package/src/Workspaces/Workspaces.js

@@ -74,7 +74,7 @@ class Workspaces {
      * @returns {AsyncIterator}
      */
     listAll (filter = null) {
-        const requestParameters = filter !== null ? { filter: filter } : {}
+        const requestParameters = filter !== null ? { filter } : {}
         return this.iterator().all(requestParameters)
     }
 
@@ -82,7 +82,7 @@ class Workspaces {
      * @returns {Page}
      */
     listFirstPage (filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.iterator().firstPage(requestParameters, pageSize)
     }
 
@@ -90,7 +90,7 @@ class Workspaces {
      * @returns {Page}
      */
     listPageAfter (afterId, filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.iterator().pageAfter(afterId, requestParameters, pageSize)
     }
 
@@ -98,7 +98,7 @@ class Workspaces {
      * @returns {Page}
      */
     listPageBefore (beforeId, filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.iterator().pageBefore(beforeId, requestParameters, pageSize)
     }
 
@@ -106,7 +106,7 @@ class Workspaces {
      * @returns {AsyncIterator}
      */
     listActive (filter = null) {
-        const requestParameters = filter !== null ? { filter: filter } : {}
+        const requestParameters = filter !== null ? { filter } : {}
         return this.activeWorkspacesIterator().all(requestParameters)
     }
 
@@ -114,7 +114,7 @@ class Workspaces {
      * @returns {Page}
      */
     listActiveFirstPage (filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.activeWorkspacesIterator().firstPage(requestParameters, pageSize)
     }
 
@@ -122,7 +122,7 @@ class Workspaces {
      * @returns {Page}
      */
     listActivePageAfter (afterId, filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.activeWorkspacesIterator().pageAfter(afterId, requestParameters, pageSize)
     }
 
@@ -130,7 +130,7 @@ class Workspaces {
      * @returns {Page}
      */
     listActivePageBefore (beforeId, filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.activeWorkspacesIterator().pageBefore(beforeId, requestParameters, pageSize)
     }
 
@@ -138,7 +138,7 @@ class Workspaces {
      * @returns {AsyncIterator}
      */
     listInactive (filter = null) {
-        const requestParameters = filter !== null ? { filter: filter } : {}
+        const requestParameters = filter !== null ? { filter } : {}
         return this.inactiveWorkspacesIterator().all(requestParameters)
     }
 
@@ -146,7 +146,7 @@ class Workspaces {
      * @returns {Page}
      */
     listInactiveFirstPage (filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.inactiveWorkspacesIterator().firstPage(requestParameters, pageSize)
     }
 
@@ -154,7 +154,7 @@ class Workspaces {
      * @returns {Page}
      */
     listInactivePageAfter (afterId, filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.inactiveWorkspacesIterator().pageAfter(afterId, requestParameters, pageSize)
     }
 
@@ -162,7 +162,7 @@ class Workspaces {
      * @returns {Page}
      */
     listInactivePageBefore (beforeId, filter = null, pageSize = null) {
-        const requestParameters = filter !== null ? { filter: filter } : null
+        const requestParameters = filter !== null ? { filter } : null
         return this.inactiveWorkspacesIterator().pageBefore(beforeId, requestParameters, pageSize)
     }
 

package/tests/accounts/changeHoldPeriod.test.js

@@ -1,7 +1,7 @@
 const testUtils = require('../testUtils.js')
 
 test('should change the holdPeriod', async () => {
-    const { client, user } = await testUtils.createTestUserAndClient()
+    const { client } = await testUtils.createTestUserAndClient()
     await client.accounts.changeHoldPeriod(14)
 
     const account = await client.accounts.retrieveMyAccount()

package/tests/accounts/changePassword.test.js

@@ -1,6 +1,6 @@
 const testUtils = require('../testUtils.js')
 
 test('should change the password', async () => {
-    const { client, user } = await testUtils.createTestUserAndClient()
+    const { client } = await testUtils.createTestUserAndClient()
     await client.accounts.changePassword('someNewPassword')
 })

package/tests/accounts/disableDraftChartDrawings.test.js

@@ -1,7 +1,7 @@
 const testUtils = require('../testUtils.js')
 
 test('should disable draft chart drawings', async () => {
-    const { client, user } = await testUtils.createTestUserAndClient()
+    const { client } = await testUtils.createTestUserAndClient()
     await client.accounts.enableDraftChartDrawings()
 
     await client.accounts.disableDraftChartDrawings()

package/tests/accounts/enableDraftChartDrawings.test.js

@@ -1,7 +1,7 @@
 const testUtils = require('../testUtils.js')
 
 test('should enable draft chart drawings', async () => {
-    const { client, user } = await testUtils.createTestUserAndClient()
+    const { client } = await testUtils.createTestUserAndClient()
     await client.accounts.disableDraftChartDrawings()
 
     await client.accounts.enableDraftChartDrawings()

package/tests/accounts/updateSettingTest.test.js

@@ -1,7 +1,7 @@
 const testUtils = require('../testUtils.js')
 
 test('updates a setting', async () => {
-    const { client, user } = await testUtils.createTestUserAndClient()
+    const { client } = await testUtils.createTestUserAndClient()
     await client.accounts.updateSetting('VALIDATE_DUPLICATE_LABELS', 'OFF')
 
     const account = await client.accounts.retrieveMyAccount()

package/tests/chartReports/chartReportSummary.test.js

@@ -94,7 +94,7 @@ test('summaryByCategoryKey', async () => {
                 seat: 16
             }
         },
-        'string11': {
+        string11: {
             count: 0,
             bySection: {},
             byObjectType: {}

package/tests/charts/addTag.test.js

@@ -1,7 +1,7 @@
 const testUtils = require('../testUtils.js')
 
 test('should add tag', async () => {
-    const { client, user } = await testUtils.createTestUserAndClient()
+    const { client } = await testUtils.createTestUserAndClient()
     const chart = await client.charts.create()
 
     await client.charts.addTag(chart.key, 'tag1')
@@ -12,7 +12,7 @@ test('should add tag', async () => {
 })
 
 test('should be able to add a tag with special characters', async () => {
-    const { client, user } = await testUtils.createTestUserAndClient()
+    const { client } = await testUtils.createTestUserAndClient()
     const chart = await client.charts.create()
 
     await client.charts.addTag(chart.key, 'tag1/:"-<>')

package/tests/charts/copyChart.test.js

@@ -1,7 +1,7 @@
 const testUtils = require('../testUtils.js')
 
 test('should copy chart', async () => {
-    const { client, user } = await testUtils.createTestUserAndClient()
+    const { client } = await testUtils.createTestUserAndClient()
     const chart = await client.charts.create('My Chart', 'BOOTHS')
 
     const copiedChart = await client.charts.copy(chart.key)