Skip to content

Posts integration#13

Merged
Bartosz-Do merged 18 commits into
mainfrom
posts-integration
Apr 4, 2026
Merged

Posts integration#13
Bartosz-Do merged 18 commits into
mainfrom
posts-integration

Conversation

@mappoya

@mappoya mappoya commented Apr 1, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@vercel

vercel Bot commented Apr 1, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
mock-data-generator Ready Ready Preview, Comment Apr 2, 2026 8:46pm

const highlightedJson = useMemo(() => {
const jsonString = JSON.stringify(data?.data ?? [], null, 2);
if (!data) return "[]";
if (!data.ok) return Prism.highlight(data.error, Prism.languages.json, "json");

@Bartosz-Do Bartosz-Do Apr 1, 2026

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.

It doesn't works ok. Hook should return error always (example: user haven't selected any field but set count to 1000). It don't have to send request to API but hook should send user-friendly responses

@Bartosz-Do Bartosz-Do 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.

still need to fix error.

const highlightedJson = useMemo(() => {
const jsonString = JSON.stringify(data?.data ?? [], null, 2);
if (!data) return "[]";
if (!data.ok) return Prism.highlight(data.error, Prism.languages.json, "json");

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.

still need to fix it

@Bartosz-Do Bartosz-Do 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.

🎸

@Bartosz-Do
Bartosz-Do merged commit 6e5ee75 into main Apr 4, 2026
3 checks passed
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