Skip to content

docs: project-tasks-list migration component review#563

Open
washyking wants to merge 5 commits intothoth-tech:mainfrom
washyking:main
Open

docs: project-tasks-list migration component review#563
washyking wants to merge 5 commits intothoth-tech:mainfrom
washyking:main

Conversation

@washyking
Copy link

Description
Added a component review markdown for ./src/app/tasks/project-tasks-list/project-tasks-list.coffee

Type of change
Documentation (update or new)

@XinHuang1112
Copy link

You did some work, but are there any interactions involved? I think you should add some Component outcomes and interactions in your component review.

@washyking
Copy link
Author

You did some work, but are there any interactions involved? I think you should add some Component outcomes and interactions in your component review.

Just updated the interactions and component outcomes section. The bottom section "Post-migration" is being left for my review and eventually the photo of the completed task @XinHuang1112

@XinHuang1112
Copy link

Good job, Alvin. I think this should be fine now.

@washyking washyking closed this Nov 19, 2024
@washyking washyking reopened this Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants