Skip to content

fix: resolve merge conflict artifacts in 4 page files (closes #256)#258

Open
Siddh2024 wants to merge 8 commits into
Nsanjayboruds:mainfrom
Siddh2024:fix/merge-conflict-artifacts
Open

fix: resolve merge conflict artifacts in 4 page files (closes #256)#258
Siddh2024 wants to merge 8 commits into
Nsanjayboruds:mainfrom
Siddh2024:fix/merge-conflict-artifacts

Conversation

@Siddh2024

Copy link
Copy Markdown
Contributor

Description

Resolves merge conflict artifacts left in 4 page files after an incomplete conflict resolution. The original conflict markers were removed but the necessary imports were lost.

Changes Made

Recommendations.jsx

  • Added back \import IssueRecommendations from '../components/IssueRecommendations'\ — was completely removed during conflict resolution

NewArrivals.jsx

  • Added \import { motion } from 'framer-motion'\ — \motion.div\ is used on lines 81-95
  • Added \import Card from '../components/Card'\ — <Card>\ component is used on line 87
  • Added \import Title from '../components/Title'\ — <Title>\ component is used on lines 47, 68

BestSellers.jsx

  • Added \import { motion } from 'framer-motion'\ — \motion.div\ is used on lines 63-77
  • Added \import Card from '../components/Card'\ — <Card>\ component is used on line 69
  • Added \import Title from '../components/Title'\ — <Title>\ component is used on lines 29, 50

NotFound.jsx

  • Already had correct imports — no changes needed

Closes #256

@Nsanjayboruds Nsanjayboruds left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

sloved the frontend lint issue


useEffect(() => {
if (!isLoading && bestsellers.length > 0 && gridRef.current) {
gsap.fromTo(

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

slove this lint issue

Comment thread frontend/src/context/ShopContext.jsx Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Unexpected console statement
Continuous Integration / Lint Frontend

Comment thread frontend/src/context/ShopContext.jsx Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Unexpected console statement
Continuous Integration / Lint Frontend


useEffect(() => {
getProducts();
}, []);

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Unexpected console statement
Continuous Integration / Lint Frontend

Comment thread frontend/src/context/ShopContext.jsx Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Unexpected console statement

@Siddh2024

Copy link
Copy Markdown
Contributor Author

@Nsanjayboruds , check this resolve

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.

[BUG] Unresolved merge conflict artifacts in 4 page files - build pipeline broken

2 participants