feat: Update environment variable naming and refactor image URL handling across the application

This commit is contained in:
2024-11-26 10:34:29 +02:00
parent 29e8d3d133
commit b7771ac827
14 changed files with 107 additions and 69 deletions

View File

@@ -58,19 +58,19 @@ jobs:
export NEXT_PUBLIC_API_BASE_URL=$(aws ssm get-parameter --name "/tasker/api/base-url" --query "Parameter.Value" --output text) export NEXT_PUBLIC_API_BASE_URL=$(aws ssm get-parameter --name "/tasker/api/base-url" --query "Parameter.Value" --output text)
export NEXT_PUBLIC_COGNITO_USER_POOL_ID=$(aws ssm get-parameter --name "/tasker/cognito/user-pool-id" --query "Parameter.Value" --output text) export NEXT_PUBLIC_COGNITO_USER_POOL_ID=$(aws ssm get-parameter --name "/tasker/cognito/user-pool-id" --query "Parameter.Value" --output text)
export NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID=$(aws ssm get-parameter --name "/tasker/cognito/client-id" --query "Parameter.Value" --output text) export NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID=$(aws ssm get-parameter --name "/tasker/cognito/client-id" --query "Parameter.Value" --output text)
export S3_PUBLIC_IMAGE_URL=$(aws ssm get-parameter --name "/tasker/s3/public-images-url" --query "Parameter.Value" --output text) export NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL=$(aws ssm get-parameter --name "/tasker/s3/public-images-url" --query "Parameter.Value" --output text)
echo "NEXT_PUBLIC_API_BASE_URL=$NEXT_PUBLIC_API_BASE_URL" >> $GITHUB_ENV echo "NEXT_PUBLIC_API_BASE_URL=$NEXT_PUBLIC_API_BASE_URL" >> $GITHUB_ENV
echo "NEXT_PUBLIC_COGNITO_USER_POOL_ID=$NEXT_PUBLIC_COGNITO_USER_POOL_ID" >> $GITHUB_ENV echo "NEXT_PUBLIC_COGNITO_USER_POOL_ID=$NEXT_PUBLIC_COGNITO_USER_POOL_ID" >> $GITHUB_ENV
echo "NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID=$NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID" >> $GITHUB_ENV echo "NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID=$NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID" >> $GITHUB_ENV
echo "S3_PUBLIC_IMAGE_URL=$S3_PUBLIC_IMAGE_URL" >> $GITHUB_ENV echo "NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL=$NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL" >> $GITHUB_ENV
- name: Set Amplify Environment Variables - name: Set Amplify Environment Variables
run: | run: |
amplify env set NEXT_PUBLIC_API_BASE_URL=$NEXT_PUBLIC_API_BASE_URL amplify env set NEXT_PUBLIC_API_BASE_URL=$NEXT_PUBLIC_API_BASE_URL
amplify env set NEXT_PUBLIC_COGNITO_USER_POOL_ID=$NEXT_PUBLIC_COGNITO_USER_POOL_ID amplify env set NEXT_PUBLIC_COGNITO_USER_POOL_ID=$NEXT_PUBLIC_COGNITO_USER_POOL_ID
amplify env set NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID=$NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID amplify env set NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID=$NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID
amplify env set S3_PUBLIC_IMAGE_URL=$S3_PUBLIC_IMAGE_URL amplify env set NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL=$NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL
- name: Deploy Amplify App - name: Deploy Amplify App
run: | run: |

View File

@@ -23,4 +23,4 @@ env:
NEXT_PUBLIC_API_BASE_URL: "" NEXT_PUBLIC_API_BASE_URL: ""
NEXT_PUBLIC_COGNITO_USER_POOL_ID: "" NEXT_PUBLIC_COGNITO_USER_POOL_ID: ""
NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID: "" NEXT_PUBLIC_COGNITO_USER_POOL_CLIENT_ID: ""
S3_PUBLIC_IMAGE_URL: "" NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL: ""

View File

@@ -1,7 +1,17 @@
import type { NextConfig } from "next"; import type { NextConfig } from "next";
const nextConfig: NextConfig = { const nextConfig: NextConfig = {
/* config options here */ images: {
remotePatterns: [
{
protocol: "https",
hostname: new URL(process.env.NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL || "")
.hostname,
port: "",
pathname: "/**",
},
],
},
}; };
export default nextConfig; export default nextConfig;

View File

@@ -1,6 +1,11 @@
import Modal from "@/app/components/Modal"; import Modal from "@/app/components/Modal";
import { Priority, Status, useCreateTaskMutation } from "@/state/api"; import {
import React, { useState } from "react"; Priority,
Status,
useCreateTaskMutation,
useGetAuthUserQuery,
} from "@/state/api";
import React, { useEffect, useState } from "react";
import { formatISO } from "date-fns"; import { formatISO } from "date-fns";
type Props = { type Props = {
@@ -22,22 +27,28 @@ const ModalNewTask = ({ isOpen, onClose, id = null }: Props) => {
const [assignedUserId, setAssignedUserId] = useState(""); const [assignedUserId, setAssignedUserId] = useState("");
const [projectId, setProjectId] = useState(""); const [projectId, setProjectId] = useState("");
const { data: currentUser } = useGetAuthUserQuery({});
const userId = currentUser?.userDetails?.userId ?? null;
useEffect(() => {
setAuthorUserId(userId || "");
}, [userId]);
const handleSubmit = async () => { const handleSubmit = async () => {
console.log(title, authorUserId, id, projectId); if (!(title && authorUserId && (id !== null || projectId))) return;
console.log("Creating task 1.."); const finalAssignedUserId = assignedUserId.trim() || authorUserId;
if (
!(title && authorUserId && assignedUserId && (id !== null || projectId))
)
return;
console.log("Creating task 2...");
const formattedStartDate = formatISO(new Date(startDate), { const formattedStartDate =
representation: "complete", startDate ??
}); formatISO(new Date(startDate), {
const formattedDueDate = formatISO(new Date(dueDate), { representation: "complete",
representation: "complete", });
}); const formattedDueDate =
dueDate ??
formatISO(new Date(dueDate), {
representation: "complete",
});
await createTask({ await createTask({
title, title,
@@ -48,16 +59,15 @@ const ModalNewTask = ({ isOpen, onClose, id = null }: Props) => {
startDate: formattedStartDate, startDate: formattedStartDate,
dueDate: formattedDueDate, dueDate: formattedDueDate,
authorUserId: authorUserId, authorUserId: authorUserId,
assignedUserId: assignedUserId, assignedUserId: finalAssignedUserId,
projectId: id !== null ? id : projectId, projectId: id !== null ? id : projectId,
}); });
onClose();
}; };
const isFormValid = () => { const isFormValid = () => {
console.log(title, authorUserId, id, projectId); return title && authorUserId && (id !== null || projectId);
return (
title && authorUserId && assignedUserId && (id !== null || projectId)
);
}; };
const selectStyles = const selectStyles =
@@ -92,9 +102,13 @@ const ModalNewTask = ({ isOpen, onClose, id = null }: Props) => {
<select <select
className={selectStyles} className={selectStyles}
value={status} value={status}
onChange={(e) => onChange={(e) => {
setStatus(Status[e.target.value as keyof typeof Status]) const selectedStatus = Object.entries(Status).find(
} ([, value]) => value === e.target.value,
)?.[0] as keyof typeof Status;
setStatus(selectedStatus ? Status[selectedStatus] : Status.ToDo);
}}
> >
<option value="">Select Status</option> <option value="">Select Status</option>
<option value={Status.ToDo}>To Do</option> <option value={Status.ToDo}>To Do</option>
@@ -139,13 +153,15 @@ const ModalNewTask = ({ isOpen, onClose, id = null }: Props) => {
onChange={(e) => setDueDate(e.target.value)} onChange={(e) => setDueDate(e.target.value)}
/> />
</div> </div>
<input {authorUserId === "" && (
type="text" <input
className={inputStyles} type="text"
placeholder="Author User ID" className={inputStyles}
value={authorUserId} placeholder="Author User ID"
onChange={(e) => setAuthorUserId(e.target.value)} value={authorUserId}
/> onChange={(e) => setAuthorUserId(e.target.value)}
/>
)}
<input <input
type="text" type="text"
className={inputStyles} className={inputStyles}

View File

@@ -77,7 +77,7 @@ const Navbar = () => {
<div className="align-center flex h-9 w-9 justify-center"> <div className="align-center flex h-9 w-9 justify-center">
{!!currentUserDetails?.profilePictureUrl ? ( {!!currentUserDetails?.profilePictureUrl ? (
<Image <Image
src={`${process.env.S3_PUBLIC_IMAGE_URL}/${currentUserDetails?.profilePictureUrl}`} src={`${process.env.NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL}/${currentUserDetails?.profilePictureUrl}`}
alt={currentUserDetails?.username || "User Profile Picture"} alt={currentUserDetails?.username || "User Profile Picture"}
width={100} width={100}
height={50} height={50}

View File

@@ -71,7 +71,7 @@ const Sidebar = () => {
</div> </div>
<div className="flex items-center gap-5 border-y-[1.5px] border-gray-200 px-8 py-4 dark:border-gray-700"> <div className="flex items-center gap-5 border-y-[1.5px] border-gray-200 px-8 py-4 dark:border-gray-700">
<Image <Image
src={`${process.env.S3_PUBLIC_IMAGE_URL}/logo.png`} src={`${process.env.NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL}/logo.png`}
alt="logo" alt="logo"
width={40} width={40}
height={40} height={40}
@@ -162,7 +162,7 @@ const Sidebar = () => {
<div className="align-center flex h-9 w-9 justify-center"> <div className="align-center flex h-9 w-9 justify-center">
{!!currentUserDetails?.profilePictureUrl ? ( {!!currentUserDetails?.profilePictureUrl ? (
<Image <Image
src={`${process.env.S3_PUBLIC_IMAGE_URL}/${currentUserDetails?.profilePictureUrl}`} src={`${process.env.NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL}/${currentUserDetails?.profilePictureUrl}`}
alt={currentUserDetails?.username || "User Profile Picture"} alt={currentUserDetails?.username || "User Profile Picture"}
width={100} width={100}
height={50} height={50}

View File

@@ -16,7 +16,7 @@ const TaskCard = ({ task }: Props) => {
<div className="flex flex-wrap"> <div className="flex flex-wrap">
{task.attachments && task.attachments.length > 0 && ( {task.attachments && task.attachments.length > 0 && (
<Image <Image
src={`${process.env.S3_PUBLIC_IMAGE_URL}/${task.attachments[0].fileURL}`} src={`${process.env.NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL}/${task.attachments[0].fileURL}`}
alt={task.attachments[0].fileName} alt={task.attachments[0].fileName}
width={400} width={400}
height={200} height={200}

View File

@@ -11,7 +11,7 @@ const UserCard = ({ user }: Props) => {
<div className="flex items-center rounded border p-4 shadow"> <div className="flex items-center rounded border p-4 shadow">
{user.profilePictureUrl && ( {user.profilePictureUrl && (
<Image <Image
src={`${process.env.S3_PUBLIC_IMAGE_URL}/${user.profilePictureUrl}`} src={`${process.env.NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL}/${user.profilePictureUrl}`}
alt="profile picture" alt="profile picture"
width={32} width={32}
height={32} height={32}

View File

@@ -24,6 +24,7 @@ const BoardView = ({ projectId, setIsModalNewTaskOpen }: BoardProps) => {
data: fetchedTasks, data: fetchedTasks,
isLoading, isLoading,
error, error,
refetch,
} = useGetTasksQuery({ projectId }); } = useGetTasksQuery({ projectId });
const [updateTaskStatus] = useUpdateTaskStatusMutation(); const [updateTaskStatus] = useUpdateTaskStatusMutation();
const [tasks, setTasks] = useState<TaskType[]>([]); const [tasks, setTasks] = useState<TaskType[]>([]);
@@ -43,6 +44,7 @@ const BoardView = ({ projectId, setIsModalNewTaskOpen }: BoardProps) => {
try { try {
await updateTaskStatus({ taskId, status: toStatus }); await updateTaskStatus({ taskId, status: toStatus });
await refetch();
} catch (error) { } catch (error) {
console.error("Failed to update task status:", error); console.error("Failed to update task status:", error);
setTasks(fetchedTasks || []); setTasks(fetchedTasks || []);
@@ -197,7 +199,7 @@ const Task = ({ task }: TaskProps) => {
> >
{task.attachments && task.attachments.length > 0 && ( {task.attachments && task.attachments.length > 0 && (
<Image <Image
src={`${process.env.S3_PUBLIC_IMAGE_URL}/${task.attachments[0].fileURL}`} src={`${process.env.NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL}/${task.attachments[0].fileURL}`}
alt={task.attachments[0].fileName} alt={task.attachments[0].fileName}
width={400} width={400}
height={200} height={200}
@@ -248,8 +250,8 @@ const Task = ({ task }: TaskProps) => {
<div className="flex -space-x-[6px] overflow-hidden"> <div className="flex -space-x-[6px] overflow-hidden">
{task.assignee && ( {task.assignee && (
<Image <Image
key={task.assignee.userId} key={`assignee#${task.assignee.userId}`}
src={`${process.env.S3_PUBLIC_IMAGE_URL}/${task.assignee.profilePictureUrl!}`} src={`${process.env.NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL}/${task.assignee.profilePictureUrl!}`}
alt={task.assignee.username} alt={task.assignee.username}
width={30} width={30}
height={30} height={30}
@@ -258,8 +260,8 @@ const Task = ({ task }: TaskProps) => {
)} )}
{task.author && ( {task.author && (
<Image <Image
key={task.author.userId} key={`author#${task.author.userId}`}
src={`${process.env.S3_PUBLIC_IMAGE_URL}/${task.author.profilePictureUrl!}`} src={`${process.env.NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL}/${task.author.profilePictureUrl!}`}
alt={task.author.username} alt={task.author.username}
width={30} width={30}
height={30} height={30}

View File

@@ -21,16 +21,19 @@ const Timeline = ({ projectId, setIsModalNewTaskOpen }: Props) => {
}); });
const ganttTasks = useMemo(() => { const ganttTasks = useMemo(() => {
if (!tasks || tasks.length === 0) return [];
return ( return (
tasks?.map((task) => ({ tasks
start: new Date(task.startDate as string), ?.filter((task) => task.startDate && task.dueDate)
end: new Date(task.dueDate as string), .map((task) => ({
name: task.title, start: new Date(task.startDate as string),
id: `Task-${task.taskId}`, end: new Date(task.dueDate as string),
type: "task" as TaskTypeItems, name: task.title,
progress: task.points ? (task.points / 10) * 100 : 0, id: `Task-${task.taskId}`,
isDisabled: false, type: "task" as TaskTypeItems,
})) || [] progress: task.points ? (task.points / 10) * 100 : 0,
isDisabled: false,
})) || []
); );
}, [tasks]); }, [tasks]);
@@ -66,16 +69,20 @@ const Timeline = ({ projectId, setIsModalNewTaskOpen }: Props) => {
</div> </div>
<div className="overflow-hidden rounded-md bg-white shadow dark:bg-dark-secondary dark:text-white"> <div className="overflow-hidden rounded-md bg-white shadow dark:bg-dark-secondary dark:text-white">
<div className="timeline"> {ganttTasks.length > 0 && (
<Gantt <div className="timeline">
tasks={ganttTasks} <Gantt
{...displayOptions} tasks={ganttTasks}
columnWidth={displayOptions.viewMode === ViewMode.Month ? 150 : 100} {...displayOptions}
listCellWidth="100px" columnWidth={
barBackgroundColor={isDarkMode ? "#101214" : "#aeb8c2"} displayOptions.viewMode === ViewMode.Month ? 150 : 100
barBackgroundSelectedColor={isDarkMode ? "#000" : "#9ba1a6"} }
/> listCellWidth="100px"
</div> barBackgroundColor={isDarkMode ? "#101214" : "#aeb8c2"}
barBackgroundSelectedColor={isDarkMode ? "#000" : "#9ba1a6"}
/>
</div>
)}
<div className="px-4 pb-5 pt-1"> <div className="px-4 pb-5 pt-1">
<button <button
className="flex items-center rounded bg-blue-primary px-3 py-2 text-white hover:bg-blue-600" className="flex items-center rounded bg-blue-primary px-3 py-2 text-white hover:bg-blue-600"

View File

@@ -31,7 +31,7 @@ const columns: GridColDef[] = [
<div className="flex h-full w-full items-center justify-center"> <div className="flex h-full w-full items-center justify-center">
<div className="h-9 w-9"> <div className="h-9 w-9">
<Image <Image
src={`${process.env.S3_PUBLIC_IMAGE_URL}/${params.value}`} src={`${process.env.NEXT_PUBLIC_S3_PUBLIC_IMAGE_URL}/${params.value}`}
alt={params.row.username} alt={params.row.username}
width={100} width={100}
height={50} height={50}

View File

@@ -1174,7 +1174,7 @@ export const handler = async () => {
console.info(`Data population for table ${tableName} complete.`); console.info(`Data population for table ${tableName} complete.`);
} }
console.log("Data population complete."); console.info("Data population complete.");
} catch (error) { } catch (error) {
console.error("Failed to populate data:", error); console.error("Failed to populate data:", error);
} }

View File

@@ -10,10 +10,9 @@ const client = new DynamoDBClient({ region: SLS_REGION });
const docClient = DynamoDBDocument.from(client); const docClient = DynamoDBDocument.from(client);
export const handler = async (event: any): Promise<any> => { export const handler = async (event: any): Promise<any> => {
console.info(`Event: ${JSON.stringify(event)}`);
const username = const username =
event.request.userAttributes["preferred_username"] || event.userName; event.request.userAttributes["preferred_username"] || event.userName;
const cognitoId = event.userName; const cognitoId = event.request.userAttributes["sub"];
const teamId = await fetchRandomTeamId(); const teamId = await fetchRandomTeamId();
try { try {

View File

@@ -17,13 +17,17 @@ export const handler = async (event: any): Promise<any> => {
const userTasks = [...authorTasks, ...assigneeTasks]; const userTasks = [...authorTasks, ...assigneeTasks];
const uniqueTasks = Array.from(
new Map(userTasks.map((task) => [task.taskId, task])).values()
);
return { return {
statusCode: 200, statusCode: 200,
headers: { headers: {
"Content-Type": "application/json", "Content-Type": "application/json",
"Access-Control-Allow-Origin": "*", "Access-Control-Allow-Origin": "*",
}, },
body: JSON.stringify(userTasks), body: JSON.stringify(uniqueTasks),
}; };
} catch (error: any) { } catch (error: any) {
return { return {