Release hotfixes to production

README.md

@@ -230,6 +230,14 @@ yarn run lint-all:fix
 
 Make sure you've set up pre-commit hooks as described above. This will ensure that linting is automatically run on staged changes before every commit.
 
+### Flower
+
+[Flower](https://flower.readthedocs.io/en/latest/) is a Celery monitoring dashboard. It is available on http://localhost:5555 after you run services:
+
+```bash
+yarn run services
+```
+
 ### Storybook
 
 Storybook is a development environment for UI components. If this is your first encounter with this tool, you can check [this presentation](https://docs.google.com/presentation/d/10JL4C9buygWsTbT62Ym149Yh9zSR9nY_ZqFumBKUY0o/edit?usp=sharing) or [its website](https://storybook.js.org/). You are encouraged to use it any time you need to develop a new UI component. It is especially suitable for smaller to middle size components that represent basic UI building blocks.

contentcuration/contentcuration/frontend/accounts/pages/Main.vue

@@ -167,7 +167,7 @@
   }
 
   .links span:not(:last-child)::after {
-    margin: 0 8px;
+    margin: 0 8px 0 12px;
     font-size: 14pt;
     color: var(--v-grey-base);
     vertical-align: middle;

contentcuration/contentcuration/frontend/administration/constants.js

@@ -1,4 +1,4 @@
-export const RouterNames = {
+export const RouteNames = {
   CHANNELS: 'CHANNELS',
   CHANNEL: 'CHANNEL',
   USERS: 'USERS',

contentcuration/contentcuration/frontend/administration/pages/AdministrationAppError.vue

@@ -26,7 +26,7 @@
 
 <script>
 
-  import { RouterNames } from '../constants';
+  import { RouteNames } from '../constants';
   import PermissionsError from 'shared/views/errors/PermissionsError';
   import ChannelNotFoundError from 'shared/views/errors/ChannelNotFoundError';
   import PageNotFoundError from 'shared/views/errors/PageNotFoundError';
@@ -61,10 +61,10 @@
       backFromCurrentPageUrl() {
         const currentPage = this.$route.name;
         let toName;
-        if (currentPage === RouterNames.USER) {
-          toName = RouterNames.USERS;
+        if (currentPage === RouteNames.USER) {
+          toName = RouteNames.USERS;
         } else {
-          toName = RouterNames.CHANNELS;
+          toName = RouteNames.CHANNELS;
         }
         return {
           to: { name: toName },

contentcuration/contentcuration/frontend/administration/pages/AdministrationIndex.vue

@@ -28,7 +28,7 @@
 
 <script>
 
-  import { RouterNames } from '../constants';
+  import { RouteNames } from '../constants';
   import AdministrationAppError from './AdministrationAppError';
   import AppBar from 'shared/views/AppBar';
   import GlobalSnackbar from 'shared/views/GlobalSnackbar';
@@ -52,12 +52,12 @@
       },
       channelsLink() {
         return {
-          name: RouterNames.CHANNELS,
+          name: RouteNames.CHANNELS,
         };
       },
       usersLink() {
         return {
-          name: RouterNames.USERS,
+          name: RouteNames.USERS,
         };
       },
     },

contentcuration/contentcuration/frontend/administration/pages/Channels/ChannelActionsDropdown.vue

@@ -119,7 +119,7 @@
 
   import { mapActions, mapGetters } from 'vuex';
   import ConfirmationDialog from '../../components/ConfirmationDialog';
-  import { RouterNames } from '../../constants';
+  import { RouteNames } from '../../constants';
   import { channelExportMixin } from 'shared/views/channel/mixins';
 
   export default {
@@ -151,7 +151,7 @@
       },
       searchChannelEditorsLink() {
         return {
-          name: RouterNames.USERS,
+          name: RouteNames.USERS,
           query: {
             search: `${this.name} ${this.channel.id}`,
           },

contentcuration/contentcuration/frontend/administration/pages/Channels/ChannelDetails.vue

@@ -54,7 +54,7 @@
 <script>
 
   import { mapActions, mapGetters } from 'vuex';
-  import { RouterNames } from '../../constants';
+  import { RouteNames } from '../../constants';
   import ChannelActionsDropdown from './ChannelActionsDropdown';
   import ChannelSharing from 'shared/views/channel/ChannelSharing';
   import Details from 'shared/views/details/Details';
@@ -109,7 +109,7 @@
       },
       backLink() {
         return {
-          name: RouterNames.CHANNELS,
+          name: RouteNames.CHANNELS,
           query: this.$route.query,
         };
       },

contentcuration/contentcuration/frontend/administration/pages/Channels/ChannelItem.vue

@@ -174,7 +174,7 @@
 
   import { mapGetters, mapActions } from 'vuex';
   import ClipboardChip from '../../components/ClipboardChip';
-  import { RouterNames } from '../../constants';
+  import { RouteNames } from '../../constants';
   import ChannelActionsDropdown from './ChannelActionsDropdown';
   import Checkbox from 'shared/views/form/Checkbox';
   import { fileSizeMixin } from 'shared/mixins';
@@ -220,7 +220,7 @@
       },
       channelModalLink() {
         return {
-          name: RouterNames.CHANNEL,
+          name: RouteNames.CHANNEL,
           params: { channelId: this.channelId },
           query: this.$route.query,
         };
@@ -230,7 +230,7 @@
       },
       searchChannelEditorsLink() {
         return {
-          name: RouterNames.USERS,
+          name: RouteNames.USERS,
           query: {
             keywords: `${this.channel.name} ${this.channelId}`,
           },

contentcuration/contentcuration/frontend/administration/pages/Channels/ChannelTable.vue

@@ -101,7 +101,7 @@
 
   import { mapGetters, mapActions } from 'vuex';
   import { tableMixin, generateFilterMixin } from '../../mixins';
-  import { RouterNames, rowsPerPageItems } from '../../constants';
+  import { RouteNames, rowsPerPageItems } from '../../constants';
   import ChannelItem from './ChannelItem';
   import { channelExportMixin } from 'shared/views/channel/mixins';
   import { routerMixin } from 'shared/mixins';
@@ -195,7 +195,7 @@
       $route: {
         deep: true,
         handler(newRoute, oldRoute) {
-          if (newRoute.name === oldRoute.name && newRoute.name === RouterNames.CHANNELS)
+          if (newRoute.name === oldRoute.name && newRoute.name === RouteNames.CHANNELS)
             this.selected = [];
         },
       },

contentcuration/contentcuration/frontend/administration/pages/Channels/__tests__/channelDetails.spec.js

@@ -1,15 +1,15 @@
 import { mount } from '@vue/test-utils';
 import router from '../../../router';
 import { factory } from '../../../store';
-import { RouterNames } from '../../../constants';
+import { RouteNames } from '../../../constants';
 import ChannelDetails from './../ChannelDetails';
 
 const store = factory();
 
 const channelId = '11111111111111111111111111111111';
 
 function makeWrapper() {
-  router.replace({ name: RouterNames.CHANNEL, params: { channelId } });
+  router.replace({ name: RouteNames.CHANNEL, params: { channelId } });
   return mount(ChannelDetails, {
     router,
     store,
@@ -37,7 +37,7 @@ describe('channelDetails', () => {
   });
   it('clicking close should close the modal', () => {
     wrapper.vm.dialog = false;
-    expect(wrapper.vm.$route.name).toBe(RouterNames.CHANNELS);
+    expect(wrapper.vm.$route.name).toBe(RouteNames.CHANNELS);
   });
   describe('load', () => {
     it('should automatically close if loadChannel does not find a channel', () => {
@@ -46,7 +46,7 @@ describe('channelDetails', () => {
         loadChannelDetails: jest.fn().mockReturnValue(Promise.resolve()),
       });
       return wrapper.vm.load().then(() => {
-        expect(wrapper.vm.$route.name).toBe(RouterNames.CHANNELS);
+        expect(wrapper.vm.$route.name).toBe(RouteNames.CHANNELS);
       });
     });
     it('load should call loadChannel and loadChannelDetails', () => {

contentcuration/contentcuration/frontend/administration/pages/Channels/__tests__/channelItem.spec.js

@@ -1,7 +1,7 @@
 import { mount } from '@vue/test-utils';
 import router from '../../../router';
 import { factory } from '../../../store';
-import { RouterNames } from '../../../constants';
+import { RouteNames } from '../../../constants';
 import ChannelItem from '../ChannelItem';
 
 const store = factory();
@@ -21,7 +21,7 @@ const channel = {
 };
 
 function makeWrapper() {
-  router.replace({ name: RouterNames.CHANNELS });
+  router.replace({ name: RouteNames.CHANNELS });
   return mount(ChannelItem, {
     router,
     store,

contentcuration/contentcuration/frontend/administration/pages/Channels/__tests__/channelTable.spec.js

@@ -1,15 +1,15 @@
 import { mount } from '@vue/test-utils';
 import router from '../../../router';
 import { factory } from '../../../store';
-import { RouterNames } from '../../../constants';
+import { RouteNames } from '../../../constants';
 import ChannelTable from '../ChannelTable';
 
 const store = factory();
 
 const loadChannels = jest.fn().mockReturnValue(Promise.resolve());
 const channelList = ['test', 'channel', 'table'];
 function makeWrapper() {
-  router.replace({ name: RouterNames.CHANNELS });
+  router.replace({ name: RouteNames.CHANNELS });
   return mount(ChannelTable, {
     router,
     store,

contentcuration/contentcuration/frontend/administration/pages/Users/UserDetails.vue

@@ -131,7 +131,7 @@
 
   import capitalize from 'lodash/capitalize';
   import { mapActions, mapGetters } from 'vuex';
-  import { RouterNames } from '../../constants';
+  import { RouteNames } from '../../constants';
   import UserStorage from './UserStorage';
   import UserActionsDropdown from './UserActionsDropdown';
   import { routerMixin, fileSizeMixin } from 'shared/mixins';
@@ -193,7 +193,7 @@
       },
       backLink() {
         return {
-          name: RouterNames.USERS,
+          name: RouteNames.USERS,
           query: this.$route.query,
         };
       },

contentcuration/contentcuration/frontend/administration/pages/Users/UserItem.vue

@@ -96,7 +96,7 @@
 
   import capitalize from 'lodash/capitalize';
   import { mapGetters } from 'vuex';
-  import { RouterNames } from '../../constants';
+  import { RouteNames } from '../../constants';
   import UserActionsDropdown from './UserActionsDropdown';
   import UserStorage from './UserStorage';
   import Checkbox from 'shared/views/form/Checkbox';
@@ -146,14 +146,14 @@
       },
       userModalLink() {
         return {
-          name: RouterNames.USER,
+          name: RouteNames.USER,
           params: { userId: this.userId },
           query: this.$route.query,
         };
       },
       searchUserChannelsLink() {
         return {
-          name: RouterNames.CHANNELS,
+          name: RouteNames.CHANNELS,
           query: {
             keywords: `${this.user.name} ${this.user.email}`,
           },

contentcuration/contentcuration/frontend/administration/pages/Users/UserTable.vue

@@ -103,7 +103,7 @@
 <script>
 
   import { mapGetters, mapActions } from 'vuex';
-  import { RouterNames, rowsPerPageItems } from '../../constants';
+  import { RouteNames, rowsPerPageItems } from '../../constants';
   import { tableMixin, generateFilterMixin } from '../../mixins';
   import EmailUsersDialog from './EmailUsersDialog';
   import UserItem from './UserItem';
@@ -195,7 +195,7 @@
       $route: {
         deep: true,
         handler(newRoute, oldRoute) {
-          if (newRoute.name === oldRoute.name && newRoute.name === RouterNames.USERS)
+          if (newRoute.name === oldRoute.name && newRoute.name === RouteNames.USERS)
             this.selected = [];
         },
       },

contentcuration/contentcuration/frontend/administration/pages/Users/__tests__/userDetails.spec.js

@@ -1,7 +1,7 @@
 import { mount } from '@vue/test-utils';
 import router from '../../../router';
 import { factory } from '../../../store';
-import { RouterNames } from '../../../constants';
+import { RouteNames } from '../../../constants';
 import UserDetails from '../UserDetails';
 
 const store = factory();
@@ -22,7 +22,7 @@ const details = {
 };
 
 function makeWrapper(userProps = {}) {
-  router.replace({ name: RouterNames.USER, params: { userId } });
+  router.replace({ name: RouteNames.USER, params: { userId } });
   return mount(UserDetails, {
     router,
     store,
@@ -53,7 +53,7 @@ describe('userDetails', () => {
   });
   it('clicking close should close the modal', () => {
     wrapper.vm.dialog = false;
-    expect(wrapper.vm.$route.name).toBe(RouterNames.USERS);
+    expect(wrapper.vm.$route.name).toBe(RouteNames.USERS);
   });
   describe('load', () => {
     it('should automatically close if loadUser does not find a channel', () => {
@@ -62,7 +62,7 @@ describe('userDetails', () => {
         loadUserDetails: jest.fn().mockReturnValue(Promise.resolve()),
       });
       return wrapper.vm.load().then(() => {
-        expect(wrapper.vm.$route.name).toBe(RouterNames.USERS);
+        expect(wrapper.vm.$route.name).toBe(RouteNames.USERS);
       });
     });
     it('load should call loadUser and loadUserDetails', () => {
@@ -79,7 +79,7 @@ describe('userDetails', () => {
     wrapper.setData({ loading: false });
     wrapper.find('[data-test="dropdown"]').vm.$emit('deleted');
     wrapper.vm.$nextTick(() => {
-      expect(wrapper.vm.$route.name).toBe(RouterNames.USERS);
+      expect(wrapper.vm.$route.name).toBe(RouteNames.USERS);
     });
   });
   it('progress bar should reflect storage used', () => {

contentcuration/contentcuration/frontend/administration/pages/Users/__tests__/userItem.spec.js

@@ -1,7 +1,7 @@
 import { mount } from '@vue/test-utils';
 import router from '../../../router';
 import { factory } from '../../../store';
-import { RouterNames } from '../../../constants';
+import { RouteNames } from '../../../constants';
 import UserItem from '../UserItem';
 
 const store = factory();
@@ -16,7 +16,7 @@ const user = {
 };
 
 function makeWrapper() {
-  router.replace({ name: RouterNames.USERS });
+  router.replace({ name: RouteNames.USERS });
   return mount(UserItem, {
     router,
     store,

contentcuration/contentcuration/frontend/administration/pages/Users/__tests__/userTable.spec.js

@@ -1,15 +1,15 @@
 import { mount } from '@vue/test-utils';
 import router from '../../../router';
 import { factory } from '../../../store';
-import { RouterNames } from '../../../constants';
+import { RouteNames } from '../../../constants';
 import UserTable from '../UserTable';
 
 const store = factory();
 
 const loadUsers = jest.fn().mockReturnValue(Promise.resolve());
 const userList = ['test', 'user', 'table'];
 function makeWrapper() {
-  router.replace({ name: RouterNames.USERS });
+  router.replace({ name: RouteNames.USERS });
   return mount(UserTable, {
     router,
     store,