Skip to content

Lap4 se 135#38

Open
S1Y333 wants to merge 3 commits into
developfrom
LAP4SE-135
Open

Lap4 se 135#38
S1Y333 wants to merge 3 commits into
developfrom
LAP4SE-135

Conversation

@S1Y333

@S1Y333 S1Y333 commented Aug 1, 2024

Copy link
Copy Markdown
Collaborator

skills share page, connect with backend, need more testing after db data updated

import CategoryDropdown from '@/src/components/CategoryDropdown/CategoryDropdown'
import SearchCard from '@/src/components/Search/Search'
import CreateProjectCard from './components/CreateProjectCard/CreateProjectCard'
//import projectData from '../../data/dummy_data_extended.json'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Don't forget to remove all unused imports

@Mich-Elle-Lo Mich-Elle-Lo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great!

@mpa-mxiang

Copy link
Copy Markdown
Collaborator

@Mich-Elle-Lo Hey, I see this PR is approved with a possible improvement to remove unused imports and a bunch of conflicts, do you mind if I step in to help?

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.

3 participants