Merge branch 'develop' into add-to-planning-refactoring

.dockerignore

@@ -0,0 +1,16 @@
+.git
+.github
+.env
+.mypy_cache
+.pytest_cache
+.vscode
+*.egg-info
+__pycache__
+dist
+env
+node_modules
+client
+e2e/node_modules
+e2e/server/env
+e2e/dist
+server/env

.github/dependabot.yml

@@ -0,0 +1,15 @@
+# To get started with Dependabot version updates, you'll need to specify which
+# package ecosystems to update and where the package manifests are located.
+# Please see the documentation for all configuration options:
+# https://docs.github.com/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file
+
+version: 2
+updates:
+  - package-ecosystem: "npm" # See documentation for possible values
+    directory: "/" # Location of package manifests
+    schedule:
+      interval: "weekly"
+  - package-ecosystem: "pip" # See documentation for possible values
+    directory: "/" # Location of package manifests
+    schedule:
+      interval: "weekly"

.github/pull_request_template.md

@@ -0,0 +1,13 @@
+## Front-end checklist
+
+<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->
+- [ ] This pull request is adding missing TypeScript types to modified code segments where it's easy to do so with confidence
+- [ ] This pull request is using TypeScript interfaces instead of prop-types and updates usages where it's quick to do so
+- [ ] This pull request is using `memo` or `PureComponent` to define new React components (and updates existing usages in modified code segments)
+- [ ] This pull request is replacing `lodash.get` with optional chaining and nullish coalescing for modified code segments
+- [ ] This pull request is not importing anything from client-core directly (use `superdeskApi`)
+- [ ] This pull request is importing UI components from `superdesk-ui-framework` and `superdeskApi` when possible instead of using ones defined in this repository.
+- [ ] This pull request is not using `planningApi` where it is possible to use `superdeskApi`
+- [ ] This pull request is not adding redux based modals
+- [ ] In this pull request, properties of redux state are not being passed as props to components; instead, we connect it to the component that needs them. Except components where using a react key is required - do not connect those due to performance reasons.
+- [ ] This pull request is not adding redux actions that do not modify state (e.g. only calling angular services; those should be moved to `planningApi`)

.github/workflows/ci-client.yml

@@ -8,19 +8,14 @@ jobs:
     strategy:
       matrix:
         node-version: [14.x]
-    env:
-      INSTALL_NODE_MODULES: true
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
-      - name: Setup Environment
-        run: ./scripts/ci-install.sh
-      - name: Start Services
-        run: ./scripts/ci-start-services.sh
+          cache: npm
       # avoid file watch limit error
       - run: echo fs.inotify.max_user_watches=524288 | sudo tee -a /etc/sysctl.conf && sudo sysctl -p
-      - run: npm list || true
-      - name: test
-        run: npm run test
+      - run: npm ci
+      - run: npm run test
+      - run: npm run lint

.github/workflows/lint-server.yml

@@ -8,6 +8,8 @@ jobs:
     steps:
     - uses: actions/checkout@v3
     - uses: actions/setup-python@v4
+      with:
+        python-version: '3.10'
     - run: pip install black~=23.0
     - run: black --check server
 
@@ -16,6 +18,8 @@ jobs:
     steps:
     - uses: actions/checkout@v3
     - uses: actions/setup-python@v4
+      with:
+        python-version: '3.10'
     - run: pip install flake8
     - run: flake8 server
 
@@ -24,5 +28,7 @@ jobs:
     steps:
     - uses: actions/checkout@v3
     - uses: actions/setup-python@v4
+      with:
+        python-version: '3.10'
     - run: pip install -Ur server/mypy-requirements.txt
     - run: mypy server

README.md

@@ -155,6 +155,10 @@ Below sections include the config options that can be defined in settings.py.
 * EVENT_EXPORT_BODY_TEMPLATE:
     * default: https://github.com/superdesk/superdesk-planning/blob/develop/server/planning/planning_export_templates.py#L39
     * Overrides the default event template used for event exports
+* EVENT_RELATED_ITEM_SEARCH_PROVIDER_NAME:
+    * Default: None
+    * Required By: Event field ``related_items`` (otherwise this field will be automatically disabled)
+    * Defines the name of the Search Provider to use for adding related items to an Event
 
 ### Planning Config
 * LONG_EVENT_DURATION_THRESHOLD:
@@ -165,6 +169,24 @@ Below sections include the config options that can be defined in settings.py.
 * PLANNING_DEFAULT_COVERAGE_STATUS_ON_INGEST:
     * Default: 'ncostat:int' - Coverage Planned
     * The default CV qcode for populating planning.coverage[x].news_coverage_status on ingest
+* DEFAULT_CREATE_PLANNING_SERIES_WITH_EVENT_SERIES
+    * Default: False
+    * If true, will default to creating series of Planning items with a recurring series of Events,
+* SYNC_EVENT_FIELDS_TO_PLANNING
+    * Default: ""
+    * Comma separated list of Planning & Coverage fields to keep in sync with the associated Event
+    * Supported Fields:
+      * slugline
+      * internal_note
+      * name
+      * place (list CVs)
+      * subject (list CVs, exclude items with scheme)
+      * custom_vocabularies (list CVs, inside subject with scheme)
+      * anpa_category (list CVs)
+      * ednote
+      * language (includes `languages` if multilingual is enabled)
+      * definition_short (copies to Planning item's `Description Text`)
+      * priority
 
 ### Assignments Config
 * SLACK_BOT_TOKEN
@@ -176,6 +198,9 @@ Below sections include the config options that can be defined in settings.py.
 * PLANNING_SEND_NOTIFICATION_FOR_SELF_ASSIGNMENT
     * Defaults to false
     * If true, sends a notification to a user on creating an assignment that is assigned to themselves
+* PLANNING_JSON_ASSIGNED_INFO_EXTENDED
+    * Defaults to `false`
+    * If `true`, it will add to planning JSON output additional info for coverages like assigned desk name/email and assigned user name/email.
 
 ### Authoring Config
 * PLANNING_CHECK_FOR_ASSIGNMENT_ON_PUBLISH

ROADMAP.md

@@ -0,0 +1,4 @@
+# Long term refactoring goals
+
+* Replace current event and planning item editor with react-based authoring component from superdesk (see `getAuthoringComponent` in `superdeskApi`)
+* Refactor `EventItem` and `PlanningItem` components so they do not connect to a redux store