placementt-core 1.400.1063 → 1.400.1064

package/lib/typeDefinitions.d.ts

@@ -1929,6 +1929,15 @@ export type ScheduledVisit = {
      * a new target type never needs a lookup table. */
     targetCollection: "placements" | "users";
     targetId: string;
+    /** The target's name, copied in at write time by whoever creates the check.
+     *
+     * Denormalised on purpose: a list of checks is a list of people, and reading the
+     * target doc per row costs one read per pupil every time a staff member opens a
+     * cohort's checks — a two-hundred-wide read for a name that never changes. Written
+     * by every generator and by the ad-hoc dialog; absent on checks written before
+     * this existed, so every reader falls back rather than showing a blank. */
+    targetForename?: string;
+    targetSurname?: string;
     date: string;
     checkType: "call" | "in-person" | "video" | "email" | "other";
     createdByUid: string;
@@ -3903,6 +3912,18 @@ export declare const CAREERS_RECORD_MODE_LABELS: Record<CareersRecordEntry["mode
  * @return {CareersRecordEntry["mode"]} the record's mode.
  */
 export declare function checkTypeToRecordMode(checkType: ScheduledVisit["checkType"]): CareersRecordEntry["mode"];
+/**
+ * Who a check is about, off the check itself.
+ *
+ * The name is denormalised onto the check precisely so a list of them needs no read
+ * per row — see ScheduledVisit.targetForename. Returns an empty string for a check
+ * written before that field existed, which is the caller's cue to fall back to
+ * whatever roster it already has rather than print a uid at a member of staff.
+ *
+ * @param {Pick<ScheduledVisit, "targetForename"|"targetSurname">} check the check.
+ * @return {string} their name, or "" when the check does not carry one.
+ */
+export declare function checkTargetName(check: Pick<ScheduledVisit, "targetForename" | "targetSurname">): string;
 export type SuggestedFixRepo = "placementt-web" | "placementt-backend" | "placementt-core";
 export type SuggestedFix = {
     isCodeChange: boolean;

package/lib/typeDefinitions.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.CAREERS_RECORD_MODE_LABELS = exports.CAREERS_RECORD_TYPE_LABELS = void 0;
 exports.checkTypeToRecordMode = checkTypeToRecordMode;
+exports.checkTargetName = checkTargetName;
 /** Staff-facing labels for each entry type. In core because the web form, the setup
  * wizard and the backend CSV export all name them and must not drift. */
 exports.CAREERS_RECORD_TYPE_LABELS = {
@@ -43,4 +44,18 @@ function checkTypeToRecordMode(checkType) {
         default: return "inPerson";
     }
 }
+/**
+ * Who a check is about, off the check itself.
+ *
+ * The name is denormalised onto the check precisely so a list of them needs no read
+ * per row — see ScheduledVisit.targetForename. Returns an empty string for a check
+ * written before that field existed, which is the caller's cue to fall back to
+ * whatever roster it already has rather than print a uid at a member of staff.
+ *
+ * @param {Pick<ScheduledVisit, "targetForename"|"targetSurname">} check the check.
+ * @return {string} their name, or "" when the check does not carry one.
+ */
+function checkTargetName(check) {
+    return `${check.targetForename ?? ""} ${check.targetSurname ?? ""}`.trim();
+}
 //# sourceMappingURL=typeDefinitions.js.map

package/lib/typeDefinitions.js.map

@@ -1 +1 @@
-{"version":3,"file":"typeDefinitions.js","sourceRoot":"","sources":["../src/typeDefinitions.ts"],"names":[],"mappings":";;;AA66HA,sDASC;AA1CD;yEACyE;AAC5D,QAAA,0BAA0B,GAA2C;IAC9E,gBAAgB,EAAE,2BAA2B;IAC7C,mBAAmB,EAAE,sBAAsB;IAC3C,cAAc,EAAE,iBAAiB;IACjC,kBAAkB,EAAE,qBAAqB;IACzC,UAAU,EAAE,oCAAoC;IAChD,aAAa,EAAE,yBAAyB;IACxC,aAAa,EAAE,wBAAwB;IACvC,KAAK,EAAE,OAAO;CACjB,CAAC;AAEW,QAAA,0BAA0B,GAA+C;IAClF,QAAQ,EAAE,WAAW;IACrB,KAAK,EAAE,YAAY;IACnB,KAAK,EAAE,OAAO;CACjB,CAAC;AAEF;;;;;;;;;;;;;GAaG;AACH,SAAgB,qBAAqB,CAAC,SAAsC;IACxE,QAAQ,SAAS,EAAE,CAAC;QACpB,KAAK,MAAM,CAAC,CAAC,OAAO,OAAO,CAAC;QAC5B,KAAK,OAAO,CAAC,CAAC,OAAO,OAAO,CAAC;QAC7B,KAAK,WAAW,CAAC;QACjB,KAAK,OAAO,CAAC;QACb,KAAK,OAAO,CAAC;QACb,OAAO,CAAC,CAAC,OAAO,UAAU,CAAC;IAC3B,CAAC;AACL,CAAC"}
+{"version":3,"file":"typeDefinitions.js","sourceRoot":"","sources":["../src/typeDefinitions.ts"],"names":[],"mappings":";;;AAs7HA,sDASC;AAaD,0CAEC;AAzDD;yEACyE;AAC5D,QAAA,0BAA0B,GAA2C;IAC9E,gBAAgB,EAAE,2BAA2B;IAC7C,mBAAmB,EAAE,sBAAsB;IAC3C,cAAc,EAAE,iBAAiB;IACjC,kBAAkB,EAAE,qBAAqB;IACzC,UAAU,EAAE,oCAAoC;IAChD,aAAa,EAAE,yBAAyB;IACxC,aAAa,EAAE,wBAAwB;IACvC,KAAK,EAAE,OAAO;CACjB,CAAC;AAEW,QAAA,0BAA0B,GAA+C;IAClF,QAAQ,EAAE,WAAW;IACrB,KAAK,EAAE,YAAY;IACnB,KAAK,EAAE,OAAO;CACjB,CAAC;AAEF;;;;;;;;;;;;;GAaG;AACH,SAAgB,qBAAqB,CAAC,SAAsC;IACxE,QAAQ,SAAS,EAAE,CAAC;QACpB,KAAK,MAAM,CAAC,CAAC,OAAO,OAAO,CAAC;QAC5B,KAAK,OAAO,CAAC,CAAC,OAAO,OAAO,CAAC;QAC7B,KAAK,WAAW,CAAC;QACjB,KAAK,OAAO,CAAC;QACb,KAAK,OAAO,CAAC;QACb,OAAO,CAAC,CAAC,OAAO,UAAU,CAAC;IAC3B,CAAC;AACL,CAAC;AAED;;;;;;;;;;GAUG;AACH,SAAgB,eAAe,CAAC,KAA6D;IACzF,OAAO,GAAG,KAAK,CAAC,cAAc,IAAI,EAAE,IAAI,KAAK,CAAC,aAAa,IAAI,EAAE,EAAE,CAAC,IAAI,EAAE,CAAC;AAC/E,CAAC"}

package/package.json

@@ -2,7 +2,7 @@
   "private": false,
   "name": "placementt-core",
   "author": "Placementt",
-  "version": "1.400.1063",
+  "version": "1.400.1064",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
   "scripts": {

package/src/typeDefinitions.ts

@@ -1918,6 +1918,15 @@ export type ScheduledVisit = {
      * a new target type never needs a lookup table. */
     targetCollection: "placements"|"users",
     targetId: string,
+    /** The target's name, copied in at write time by whoever creates the check.
+     *
+     * Denormalised on purpose: a list of checks is a list of people, and reading the
+     * target doc per row costs one read per pupil every time a staff member opens a
+     * cohort's checks — a two-hundred-wide read for a name that never changes. Written
+     * by every generator and by the ad-hoc dialog; absent on checks written before
+     * this existed, so every reader falls back rather than showing a blank. */
+    targetForename?: string,
+    targetSurname?: string,
     date: string,                       // yyyy-mm-dd
     checkType: "call"|"in-person"|"video"|"email"|"other",
     createdByUid: string,
@@ -4022,6 +4031,21 @@ export function checkTypeToRecordMode(checkType: ScheduledVisit["checkType"]): C
     }
 }
 
+/**
+ * Who a check is about, off the check itself.
+ *
+ * The name is denormalised onto the check precisely so a list of them needs no read
+ * per row — see ScheduledVisit.targetForename. Returns an empty string for a check
+ * written before that field existed, which is the caller's cue to fall back to
+ * whatever roster it already has rather than print a uid at a member of staff.
+ *
+ * @param {Pick<ScheduledVisit, "targetForename"|"targetSurname">} check the check.
+ * @return {string} their name, or "" when the check does not carry one.
+ */
+export function checkTargetName(check: Pick<ScheduledVisit, "targetForename"|"targetSurname">): string {
+    return `${check.targetForename ?? ""} ${check.targetSurname ?? ""}`.trim();
+}
+
 export type SuggestedFixRepo = "placementt-web"|"placementt-backend"|"placementt-core";
 
 export type SuggestedFix = {