Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

6 Render a project board with columns and tasks #29

Merged
merged 6 commits into from
Dec 2, 2023

Conversation

joh-ann
Copy link
Contributor

@joh-ann joh-ann commented Dec 2, 2023

Summary:

This adds a ProjectBoard component that based on a selected project, it will render the proper project tasks into 3 columns: Backlog, Doing, and Done

Type of change

  • New feature
  • Bug fix
  • Refactor
  • Testing
  • Comment:

Checklist:

  • Code needs to be tested
  • All tests are passing
  • Create new branch AFTER you merge
  • Whats next: Might have to do some cleaning up/refactoring with the current code
image

@joh-ann joh-ann self-assigned this Dec 2, 2023
Copy link

vercel bot commented Dec 2, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
analog-fe ✅ Ready (Inspect) Visit Preview Dec 2, 2023 9:17am

<Project
key={project.id}
title={project.title}
onClick={() => setSelectedProject(project)}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this work?

import Column from "../Column/Column";
import Task from "../Task/Task";

function ProjectBoard({ selectedProject }) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So overall I see what you're doing and this is great! I wonder if we should really be manipulating GS for the selected project though? I'm sure I'm the one that originally had projects being passed down as props

function ProjectBoard({ selectedProject }) {
const user = useSelector((state) => state.user.user);

const projectTasks = selectedProject ? selectedProject.tasks : [];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You had to do this because GS is not cognizant of what the selected project is

return (
<>
<div className="flex flex-col w-10/12">
{selectedProject ? <p>{selectedProject.title}</p> : <p>No project selected</p>}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sick

@loganpaulmatheny loganpaulmatheny merged commit aac3da6 into main Dec 2, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants