Skip to content

Commit

Permalink
fix(article): solved the issue of not being exposed
Browse files Browse the repository at this point in the history
  • Loading branch information
gracefulBrown committed May 24, 2024
1 parent e37c063 commit 1f03fd5
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 23 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -85,14 +85,7 @@ public void likeArticle(final Long id, final LoginMember loginMember, final Bool
public List<ArticleResponse> getFilteredArticles(final LoginMember member,
final ArticleFilterType course,
final boolean onlyBookmarked) {
if (member.isMember()) {
return articleRepository.findArticlesByCourseAndMember(course.getPartName(),
member.getId(), onlyBookmarked).stream()
.map(article -> ArticleResponse.of(article, member.getId()))
.collect(toList());
}

return articleRepository.findArticlesByCourse(course.getPartName()).stream()
return articleRepository.findAllByOrderByCreatedAtDesc().stream()
.map(article -> ArticleResponse.of(article, member.getId()))
.collect(toList());
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ public interface ArticleRepository extends JpaRepository<Article, Long> {
Optional<Article> findFetchLikeById(@Param("id") final Long id);

@Query("SELECT DISTINCT a FROM Article a " +
"JOIN DepartmentMember dm ON dm.member.id = dm.member.id " +
"JOIN DepartmentMember dm ON a.member.id = dm.member.id " +
"JOIN dm.department d " +
"WHERE d.part = :course " +
"ORDER by a.createdAt desc")
Expand Down
28 changes: 14 additions & 14 deletions frontend/src/pages/ArticleListPage/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -56,20 +56,20 @@ const ArticleListPage = () => {
return (
<div css={[MainContentStyle]}>
<Container>
<FilteringWrapper>
<SelectBoxWrapper>
<SelectBox
isClearable={false}
value={selectedCourse}
defaultOption={selectedCourse}
options={CATEGORY_OPTIONS}
onChange={changeFilterOption}
/>
</SelectBoxWrapper>
{isLoggedIn && (
<ArticleBookmarkFilter checked={checked} handleCheckBookmark={handleCheckBookmark} />
)}
</FilteringWrapper>
{/*<FilteringWrapper>*/}
{/* <SelectBoxWrapper>*/}
{/* <SelectBox*/}
{/* isClearable={false}*/}
{/* value={selectedCourse}*/}
{/* defaultOption={selectedCourse}*/}
{/* options={CATEGORY_OPTIONS}*/}
{/* onChange={changeFilterOption}*/}
{/* />*/}
{/* </SelectBoxWrapper>*/}
{/* {isLoggedIn && (*/}
{/* <ArticleBookmarkFilter checked={checked} handleCheckBookmark={handleCheckBookmark} />*/}
{/* )}*/}
{/*</FilteringWrapper>*/}
{authorized && (
<Button
type="button"
Expand Down

0 comments on commit 1f03fd5

Please sign in to comment.