fix: schema updates (#953)

- cherry-pick migration from IN-831-edit-organization-page
- update bool defaults

<!--- Please provide a general summary of your changes in the title
above -->

# Pull Request type

<!-- Please try to limit your pull request to one type; submit multiple
pull requests if needed. -->

Please check the type of change your PR introduces:

- [x] Bugfix
- [ ] Feature
- [ ] Code style update (formatting, renaming)
- [ ] Refactoring (no functional changes, no API changes)
- [ ] Build-related changes
- [ ] Documentation content changes
- [ ] Other (please describe):

## What is the current behavior?

<!-- Please describe the current behavior that you are modifying, or
link to a relevant issue. -->

Issue Number: N/A

## What is the new behavior?

<!-- Please describe the behavior or changes that are being added by
this PR. -->

OrgLocation api query was excluding results due to boolean value allowed
to be null.

## Does this introduce a breaking change?

- [ ] Yes
- [ ] No

<!-- If this does introduce a breaking change, please describe the
impact and migration path for existing applications below. -->

## Other information

<!-- Any other information that is important to this PR, such as
screenshots of how the component looks before and after the change. -->

packages/api/router/orgHours/query.forHoursDisplay.handler.ts

@@ -32,6 +32,7 @@ export const forHoursDisplay = async ({ input }: TRPCHandlerParams<TForHoursDisp
 		select: { id: true, dayIndex: true, start: true, end: true, closed: true, tz: true },
 		orderBy: [{ dayIndex: 'asc' }, { start: 'asc' }],
 	})
+	if (!result.length) return null
 
 	const { weekYear, weekNumber } = DateTime.now()
 	const intervalResults = result.map(({ start, end, tz, dayIndex, ...rest }) => {
@@ -42,7 +43,7 @@ export const forHoursDisplay = async ({ input }: TRPCHandlerParams<TForHoursDisp
 				weekNumber,
 			}),
 			DateTime.fromJSDate(end, { zone: tz ?? 'America/New_York' }).set({
-				weekday: dayIndex,
+				weekday: start > end ? dayIndex + 1 : dayIndex,
 				weekYear,
 				weekNumber,
 			})

packages/db/prisma/migrations/20230928155156_/migration.sql

@@ -0,0 +1,2 @@
+-- AlterTable
+ALTER TABLE "OrgPhone" ADD COLUMN     "countryCode" TEXT;

packages/db/prisma/migrations/20231211210937_boolean_defaults_for_location/migration.sql

@@ -0,0 +1,37 @@
+/*
+ Warnings:
+
+ - Made the column `mailOnly` on table `OrgLocation` required. This step will fail if there are existing NULL values in that column.
+ - Made the column `mapCityOnly` on table `OrgLocation` required. This step will fail if there are existing NULL values in that column.
+ - Made the column `notVisitable` on table `OrgLocation` required. This step will fail if there are existing NULL values in that column.
+ */
+-- Update values for Location
+UPDATE
+	"OrgLocation"
+SET
+	"mailOnly" = FALSE
+WHERE
+	"mailOnly" IS NULL;
+
+UPDATE
+	"OrgLocation"
+SET
+	"mapCityOnly" = FALSE
+WHERE
+	"mapCityOnly" IS NULL;
+
+UPDATE
+	"OrgLocation"
+SET
+	"notVisitable" = FALSE
+WHERE
+	"notVisitable" IS NULL;
+
+-- AlterTable
+ALTER TABLE "OrgLocation"
+	ALTER COLUMN "mailOnly" SET NOT NULL,
+	ALTER COLUMN "mailOnly" SET DEFAULT FALSE,
+	ALTER COLUMN "mapCityOnly" SET NOT NULL,
+	ALTER COLUMN "mapCityOnly" SET DEFAULT FALSE,
+	ALTER COLUMN "notVisitable" SET NOT NULL,
+	ALTER COLUMN "notVisitable" SET DEFAULT FALSE;

packages/db/prisma/schema.prisma

@@ -523,6 +523,7 @@ model OrgPhone {
   id              String   @id @default(cuid())
   legacyId        String?  @unique /// old ID from MongoDB
   legacyDesc      String?
+  countryCode     String?
   number          String
   ext             String?
   primary         Boolean  @default(false)
@@ -630,9 +631,9 @@ model OrgLocation {
   postCode String?
   primary  Boolean @default(false)
 
-  mailOnly     Boolean?
-  notVisitable Boolean?
-  mapCityOnly  Boolean?
+  mailOnly     Boolean @default(false)
+  notVisitable Boolean @default(false)
+  mapCityOnly  Boolean @default(false)
 
   description   FreeText? @relation(fields: [descriptionId], references: [id], onDelete: SetNull, onUpdate: Cascade)
   descriptionId String?   @unique

packages/ui/components/data-display/Hours.tsx

@@ -1,4 +1,4 @@
-import { createStyles, List, rem, Stack, Table, Text, Title } from '@mantine/core'
+import { createStyles, List, rem, Skeleton, Stack, Table, Text, Title } from '@mantine/core'
 import { Interval } from 'luxon'
 import { useTranslation } from 'next-i18next'
 
@@ -17,36 +17,58 @@ const useStyles = createStyles(() => ({
 	dow: {
 		verticalAlign: 'baseline',
 		paddingRight: rem(4),
+		borderTopStyle: 'none !important' as 'none',
+		paddingTop: '0 !important',
+		paddingBottom: '0 !important',
+	},
+	hours: {
+		borderTopStyle: 'none !important' as 'none',
+		paddingTop: '0 !important',
+		paddingBottom: '0 !important',
 	},
 }))
 
+const nullObj = {
+	0: [],
+	1: [],
+	2: [],
+	3: [],
+	4: [],
+	5: [],
+	6: [],
+}
+
 export const Hours = ({ parentId, label = 'regular' }: HoursProps) => {
 	const { t, i18n } = useTranslation('common')
 	const variants = useCustomVariant()
 	const { classes } = useStyles()
-	const { data } = api.orgHours.forHoursDisplay.useQuery(parentId)
+	const { data, isLoading } = api.orgHours.forHoursDisplay.useQuery(parentId)
 	const dayMap = useLocalizedDays(i18n.resolvedLanguage)
-	if (!data) return null
+	if (!data && !isLoading) return null
 
 	const labelKey = labelKeys[label]
 	const timezone: string | null = null
 
-	const hourTable = Object.entries(data).map(([dayIdx, data]) => {
+	const hourTable = Object.entries(data ?? nullObj).map(([dayIdx, data]) => {
 		return (
 			<tr key={dayIdx}>
-				<td className={classes.dow}>{dayMap.get(parseInt(dayIdx))}</td>
-				<td>
+				<td className={classes.dow}>
+					<Text>{dayMap.get(parseInt(dayIdx))}</Text>
+				</td>
+				<td className={classes.hours}>
 					<List listStyleType='none'>
 						{data.map(({ id, interval: intervalISO, closed }) => {
 							const interval = Interval.fromISO(intervalISO)
 
 							return (
 								<List.Item key={id}>
-									{closed
-										? t('hours.closed')
-										: interval.toDuration('hours').valueOf() === OPEN_24_MILLISECONDS
-											? t('hours.open24')
-											: interval.toFormat('hh:mm a')}
+									<Text>
+										{closed
+											? t('hours.closed')
+											: interval.toDuration('hours').valueOf() === OPEN_24_MILLISECONDS
+												? t('hours.open24')
+												: interval.toFormat('hh:mm a')}
+									</Text>
 								</List.Item>
 							)
 						})}
@@ -58,13 +80,15 @@ export const Hours = ({ parentId, label = 'regular' }: HoursProps) => {
 
 	return (
 		<Stack spacing={12}>
-			<div>
-				<Title order={3}>{t(labelKey)}</Title>
-				<Text variant={variants.Text.utility4darkGray}>{timezone}</Text>
-			</div>
-			<Table>
-				<tbody>{hourTable}</tbody>
-			</Table>
+			<Skeleton visible={isLoading}>
+				<div>
+					<Title order={3}>{t(labelKey)}</Title>
+					<Text variant={variants.Text.utility4darkGray}>{timezone}</Text>
+				</div>
+				<Table>
+					<tbody>{hourTable}</tbody>
+				</Table>
+			</Skeleton>
 		</Stack>
 	)
 }