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.

Sign up for GitHub

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

Jump to bottom

Implement article page with dynamic routing #297

Draft
wants to merge 32 commits into
base: main
Choose a base branch
from
Draft

Implement article page with dynamic routing #297

wants to merge 32 commits into from

Conversation

julianaijal
Copy link
Owner

No description provided.

@vercel Vercel
Copy link

vercel bot commented Nov 8, 2024 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
hallo-met-mij ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 10, 2024 11:56pm

@julianaijal
update fetchPosts import and usage to handle additional article api f…
830053a 
…etch
@julianaijal
add dynamic page in slug dir
a12882e
@julianaijal
add function to get article info by slug
57f3206
@julianaijal
add to do comment
d99a962
@julianaijal
add another to do comment
bd4570d
@julianaijal
remove duplicate dynamic route to fix build
9b625ef
@julianaijal
resolve params props to fix build
ddf906c
@julianaijal
Add skeleton for helper function to handle graphql requests
83f8298
@julianaijal
add catch error
79dc259
@julianaijal
add response error logs
1e7a62d
@julianaijal
add error log if resp data is not working
13c318c
@julianaijal
test new api function
5becd88
@julianaijal
confirm console log returns new api function
0b24c24
@julianaijal
refactor api calls to use centralized util function
d75b8d2
@julianaijal
fetch slugs for dynamic route list
969ab99
@julianaijal
pass props to child components
4d794e4
@julianaijal
pass slug to create dynamic page
93d387b
@julianaijal
add temp hardcoded img till setup in hygraph
38e4da4
@julianaijal
get article info by slug
e45f9a8
@julianaijal
render props
f7f1334
@julianaijal
add html with some safety stuff
d5c8569
@julianaijal
move const out of error handling
31e4ad0
@julianaijal
move back fetching and sanitization to error handling because yea we …
0ed6c05 
…need to know if it works on all steps
@julianaijal
remove await since not a Promise
bf10f92
@julianaijal
another promise fix since previous one still failed build
f25ef2a
@julianaijal
move interface
3b870d9
@julianaijal
install packages
1e9998c
@julianaijal
improve width comparision with safer fallback
1e3c768
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@julianaijal