BCAP Messaging backend by seeker25 · Pull Request #1611 · bcgov/nr-bcap

seeker25 · 2026-07-09T19:40:59Z

bcgov/arches-zod-validation#3 <-- ideally needs to get merged as well

I didn't try the dashboard counts beyond just loading them - I think they are covered in unit tests though. We should have a function that will work for counts on a process requirement basis.

seeker25 · 2026-07-09T19:42:07Z

+CONTRIBUTOR_TYPES = [
+    # (item id, prefLabel value id, label, sortorder, concept uri)
+    (
+        "8c4e2933-a97a-5f47-89e7-78d1d8aaad81",
+        "c0b10d00-fbfa-58c3-ba46-3dc39d61a87e",
+        "Individual",
+        0,
+        "http://bcap/bcap8c4e2933-a97a-5f47-89e7-78d1d8aaad81",
+    ),
+    (
+        "fd2053d3-c010-519a-97e5-52f3c7b508e2",
+        "891278e4-5d67-571a-a0c4-414896d36d1c",
+        "Organization",
+        1,
+        "http://bcap/bcapfd2053d3-c010-519a-97e5-52f3c7b508e2",
+    ),
+]


had to add these - we don't have pkg file that load them in

added some comments as per Brett's suggestion for this

seeker25 · 2026-07-09T19:48:05Z

-        z.iso.date(),
-        z.iso.datetime({ offset: true, local: true })
-    ]).nullable()
+    node_value: z.string().regex(/^\d{4}-\d{2}-\d{2}([ T]\d{2}:\d{2}:\d{2}([+-]\d{2}:\d{2}|Z)?)?$/).nullable()


Had to do this, arches dates aren't saved as ISO - there is a way to override that but it impacts other dates/datetimes

if I try to change how arches saves the date it can impact other dates in the system, so I've opted to just make this a bit more flexible on the frontend

seeker25 · 2026-07-09T19:48:29Z

+def register_offset_preserving_date_field():
+    """Use the offset-preserving field for date nodes on the REST write path.
+    Call from AppConfig.ready(): the imports must be deferred until the app
+    registry is loaded."""
+    from django.db.models import DateTimeField
+    from arches_querysets.rest_framework.serializers import TileAliasedDataSerializer
+
+    TileAliasedDataSerializer.register_custom_datatype_field(
+        DateTimeField, OffsetPreservingDateTimeField
+    )


Arches queryset problem I'll create a PR if I have some time

P-Hansen

LGTM

seeker25 · 2026-07-10T13:10:12Z

we may need a way to get counts not on a dashboard route, but we'll see as you implement

seeker25 · 2026-07-10T13:15:30Z

+    def submission_context_ids_for_permits(self, permits, requirements_by_permit=None):
+        """Map each permit to the resource ids its unread counts span (the permit
+        and its requirements' submission hosts); pass known requirement ids to
+        skip a query."""
+        permit_ids = [str(permit.pk) for permit in permits]
+        contexts = {pk: {pk} for pk in permit_ids}
+
+        if requirements_by_permit is None:
+            requirements_by_permit = self._requirements.requirement_ids_by_permit(
+                permit_ids
+            )
+        all_requirement_ids = set(chain.from_iterable(requirements_by_permit.values()))
+        hosts_by_requirement = self._requirements.host_ids_by_requirement(
+            all_requirement_ids
+        )
+        for permit_id, requirement_ids in requirements_by_permit.items():
+            for requirement_id in requirement_ids:
+                hosts = hosts_by_requirement.get(requirement_id, ())
+                contexts[permit_id].update(hosts)
+        return contexts


we might need this done at the module level as well (groupings of process requirements)

next PR or another PR

seeker25 · 2026-07-10T13:17:02Z

-def _date_node_value(max_length):
-    # Accept a bare date or a full datetime; anyOf becomes a zod union.
-    return {
-        "nullable": True,
-        "anyOf": [
-            {"type": "string", "format": "date"},
-            {"type": "string", "format": "date-time"},
-        ],
-    }


Changed this a bit and moved to zod-validation library. It would work for createdon (on the model) but not for message_created_on (inside the json)

seeker25 · 2026-07-10T13:18:18Z

+<script setup lang="ts">
+import { reactive } from 'vue';
+import { z } from 'zod';
+import arches from 'arches';
+import {
+    zApiPermitApplicationCreateResponse,
+    zBcapMessage,
+    zBcapMessageWritable,
+    zPaginatedBcapMessageList,
+    zPaginatedContributorList,
+} from '@/bcap/client/zod.gen.ts';
+import { formatDateTime, getCsrfToken, getDisplayValue } from '@/bcap/util.ts';
+import type { AliasedNodeData } from '@/arches_component_lab/types.ts';
+


we'll need to gun this file when Philip is done with it

seeker25 · 2026-07-10T13:18:47Z

+    {
+        id: 'site-visit',
+        label: 'Site Visit',
+        subtitle: 'Coming soon',
+        icon: 'fa fa-location-dot',
+        disabled: true,
+    },


Site visit as requested in the menu / tiles at the end

seeker25 · 2026-07-10T13:19:20Z

                return False
            tile.data[username_node] = username
            tile.save()
+        Resource.objects.get(pk=contributor_id).index()


we were missing this, we may need to do a simple python script to reindex some users =(

seeker25 · 2026-07-10T13:20:03Z

+        path: arches.urls.plugin('external-permit-workflows/message-demo'),
+        name: 'messageDemo',
+        component: () =>
+            import('@/bcap/apps/Permit/components/MessageDemo.vue'),
+        meta: {
+            shouldShowNavigation: true,
+            requiresAuthentication: true,
+        },
+    },


Remove, this is temp

seeker25 · 2026-07-10T13:21:41Z

+def is_internal_user(user):
+    """True if the user is ministry staff (a superuser or holds an internal
+    role); everyone else is treated as an external applicant."""
+    if not getattr(user, "is_authenticated", False):
+        return False
+    if getattr(user, "is_superuser", False):
+        return True
+    return user.groups.filter(name__in=INTERNAL_ROLES).exists()


I think replacing this with granting group access will replace this in the future with Guardian

seeker25 · 2026-07-10T13:43:57Z

Might need some url clean up next PR

api/bcap_message/<uuid:pk>/
vs
api/bcap_message/<uuid:pk>

If you are using Django or Django REST Framework (DRF), the trailing slash is the strict default. Django uses a setting called APPEND_SLASH = True.

If a client hits api/requirement_submission/<uuid:pk>, Django will automatically return a 301 Moved Permanently redirect to api/requirement_submission/<uuid:pk>/.

…ssageWritable to fix type issues

bferguso

Nice work! This is a great push forward.

If you have time next week lets try to move the queryset patch into the arches-querysets so we don't forget about it and start to drift.

BCAP Messaging backend base commit

df3b5d1

seeker25 commented Jul 9, 2026

View reviewed changes

seeker25 requested review from P-Hansen and bferguso July 9, 2026 19:46

seeker25 commented Jul 9, 2026

View reviewed changes

seeker25 added 3 commits July 9, 2026 12:54

fix for unit test database

d3aceb7

update toml and test_settings

93860b6

lint

faee169

seeker25 changed the title ~~BCAP Messaging backend base commit~~ BCAP Messaging backend Jul 9, 2026

P-Hansen approved these changes Jul 9, 2026

View reviewed changes

seeker25 commented Jul 10, 2026

View reviewed changes

Comment thread pyproject.toml Outdated

seeker25 added 4 commits July 10, 2026 07:24

Clean up routes and add in read date PATCH

16643d5

DRF pairs GET and PATCH together intentionally, use the PatchedBCAPMe…

8fd317a

…ssageWritable to fix type issues

move back to main

b175497

Add comment and fix up check for migration

b35cb25

bferguso approved these changes Jul 10, 2026

View reviewed changes

bferguso merged commit b9f7f95 into release/2.0.x Jul 10, 2026
8 of 9 checks passed

bferguso deleted the ts/feat/1593_bcap_messaging_backend branch July 10, 2026 15:46

Uh oh!

Conversation

seeker25 commented Jul 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

P-Hansen left a comment

Choose a reason for hiding this comment

Uh oh!

seeker25 commented Jul 10, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

seeker25 Jul 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

seeker25 Jul 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

seeker25 commented Jul 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

bferguso left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

seeker25 commented Jul 9, 2026 •

edited

Loading

seeker25 Jul 10, 2026 •

edited

Loading

seeker25 Jul 10, 2026 •

edited

Loading

seeker25 commented Jul 10, 2026 •

edited

Loading