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.

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

problem #1210

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

problem #1210

Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
129 changes: 88 additions & 41 deletions src/App.tsx
Original file line number Diff line number Diff line change
@@ -1,60 +1,107 @@
import classNames from 'classnames';

import 'bulma/css/bulma.css';
import '@fortawesome/fontawesome-free/css/all.css';
import './App.scss';

import { PostsList } from './components/PostsList';
import { PostDetails } from './components/PostDetails';
import { UserSelector } from './components/UserSelector';
import { useEffect, useState } from 'react';
import { User } from './types/User';
import { getUsers } from './api/users';
import { Loader } from './components/Loader';
import { PostsList } from './components/PostsList/PostsList';
import { Post } from './types/Post';
import { getPosts } from './api/posts';
import { PostDetails } from './components/PostDetails';
import classNames from 'classnames';

Choose a reason for hiding this comment

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

Ensure that the PostDetails component is correctly implemented and functional, as it is crucial for displaying the selected post details in the sidebar. Verify that this component exists and is correctly imported.


export const App = () => (
<main className="section">
<div className="container">
<div className="tile is-ancestor">
<div className="tile is-parent">
<div className="tile is-child box is-success">
<div className="block">
<UserSelector />
</div>
export const App = () => {
const [users, setUsers] = useState<User[]>([]);
const [posts, setPosts] = useState<Post[]>([]);
const [selectedPost, setSelectedPost] = useState<Post | null>(null);
const [notification, setNotification] = useState('');
const [isLoading, setIsLoading] = useState(false);

<div className="block" data-cy="MainContent">
<p data-cy="NoSelectedUser">No user selected</p>
// #region load
useEffect(() => {
getUsers().then(setUsers);
}, []);
// #endregion

<Loader />
const handleLoadPosts = async (userId: number) => {
setIsLoading(true);
setPosts([]);
setNotification('');
setSelectedPost(null);

<div
className="notification is-danger"
data-cy="PostsLoadingError"
>
Something went wrong!
</div>
try {
const loadedPosts = await getPosts(userId);

<div className="notification is-warning" data-cy="NoPostsYet">
No posts yet
if (loadedPosts.length === 0) {
setNotification('is-warning');
}

setPosts(loadedPosts);
} catch {
setNotification('is-danger');
} finally {
setIsLoading(false);
}
};

return (
<main className="section">
<div className="container">
<div className="tile is-ancestor">
<div className="tile is-parent">
<div className="tile is-child box is-success">
<div className="block">
<UserSelector users={users} onSelect={handleLoadPosts} />
</div>

<PostsList />
<div className="block" data-cy="MainContent">
{!isLoading && posts.length === 0 && !notification && (
<p data-cy="NoSelectedUser">No user selected</p>
)}
{!isLoading && posts.length > 0 && (
<PostsList posts={posts} onSelect={setSelectedPost} />
)}
{isLoading && <Loader />}

{notification === 'is-danger' && (
<div
className="notification is-danger"
data-cy="PostsLoadingError"
>
Something went wrong!
</div>
)}

{notification === 'is-warning' && (
<div className="notification is-warning" data-cy="NoPostsYet">
No posts yet
</div>
)}
</div>
</div>
</div>
</div>

<div
data-cy="Sidebar"
className={classNames(
'tile',
'is-parent',
'is-8-desktop',
'Sidebar',
'Sidebar--open',
)}
>
<div className="tile is-child box is-success ">
<PostDetails />
<div
data-cy="Sidebar"
className={classNames(
'tile',
'is-parent',
'is-8-desktop',
'Sidebar',
{ 'Sidebar--open': selectedPost },
)}
>
{selectedPost && (
<div className="tile is-child box is-success">
<PostDetails post={selectedPost} />
</div>
)}
</div>
</div>
</div>
</div>
</main>
);
</main>
);
};
14 changes: 14 additions & 0 deletions src/api/comments.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
import { Comment } from '../types/Comment';
import { client } from '../utils/fetchClient';

export const getComments = (postId: number) => {
return client.get<Comment[]>(`/comments?postId=${postId}`);
};

export const deleteComment = (commentId: number) => {
return client.delete(`/comments/${commentId}`);
};

export const addComment = ({ name, email, body, postId }: Comment) => {
return client.post<Comment>(`/comments`, { name, email, body, postId });
};
6 changes: 6 additions & 0 deletions src/api/posts.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
import { Post } from '../types/Post';
import { client } from '../utils/fetchClient';

export const getPosts = (userId: number) => {
return client.get<Post[]>(`/posts?userId=${userId}`);
};
6 changes: 6 additions & 0 deletions src/api/users.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
import { User } from '../types/User';
import { client } from '../utils/fetchClient';

export const getUsers = () => {
return client.get<User[]>(`/users`);
};
103 changes: 0 additions & 103 deletions src/components/NewCommentForm.tsx

This file was deleted.

Loading
Loading