Skip to content

Frontend/job details page - #161

Merged
Kaylahray merged 30 commits into
artisyn-io:mainfrom
akprinciple:frontend/job-details-page
Jun 4, 2026
Merged

Frontend/job details page#161
Kaylahray merged 30 commits into
artisyn-io:mainfrom
akprinciple:frontend/job-details-page

Conversation

@akprinciple

Copy link
Copy Markdown
Contributor

feat(frontend): add job details page

closes #88

Description

This PR introduces a dedicated dynamic route for the job details page. It allows artisans to click on a specific job from the listings and view comprehensive information about the job before deciding to apply.

User Story

As an artisan,
I want to inspect full job details,
so that I can make informed apply decisions.

Changes Made

  • New Route Added: Created the dynamic route at src/app/(dashboard)/artisan/jobs/[id]/page.tsx.
  • Dynamic Fetching: Implemented Next.js 16 async route params handling to fetch job data dynamically based on the URL id.
  • Case-Insensitive Lookup: Added resilient ID matching using decodeURIComponent and .toLowerCase() to handle edge cases like spaces and case mismatches.
  • UI Implementation: Built a detailed UI layout displaying the job's title, category, budget, urgency, location, full requirements, and an "Apply now" Call-to-Action.
  • Error Handling: Integrated Next.js notFound() to gracefully handle invalid or missing job IDs by rendering a 404 state.

Acceptance Criteria

  • Dynamic job detail route is available.
  • Invalid id renders not found state.

Notes for Reviewers

Currently, the data is being fetched from the dummyjobs.ts file. The "Apply now" button is intentionally left without an onClick handler as this is a Server Component; it will be extracted into a Client Component in a future PR once the application flow is implemented.

@Kaylahray Kaylahray 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.

Hello @akprinciple Please resolve conflicts n workflow

@akprinciple

Copy link
Copy Markdown
Contributor Author

@Kaylahray Conflict resolved

@Kaylahray

Copy link
Copy Markdown
Contributor

@akprinciple Please help fix, Left u a message on the other PR

@akprinciple

Copy link
Copy Markdown
Contributor Author

@Kaylahray fixed

@Kaylahray

Kaylahray commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Please run pnpm build and pnpm lint, your workflow fails @akprinciple on both Prs

@Kaylahray
Kaylahray merged commit cce7fb1 into artisyn-io:main Jun 4, 2026
1 check failed
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.

Job details page

2 participants