[Frontend] [Feat]: Display post list with result from searching keyword #567
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
badukwei
commented
Oct 31, 2024
| const client = this.getClient() | ||
| const response = await client.get<FetchPostsResponse>( | ||
| `/post?page=${page}`, | ||
| ) | ||
| //TODO: After relay update change to |
There was a problem hiding this comment.
@ryanycw 目前前端的更新大概完成了，這邊我看目前 fetchPost api 後端還沒有關鍵字搜尋的功能，如果更新好，只需要改成這段應該就可以用了。
|
另外我想提一下，因為搜尋的 routing 結構通常會跟原本主頁的分開，因為可能會使用不同的排版或是元件，但目前因為都是使用同樣的 ui 所以我沒有再新增不同的路徑。 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
新增搜尋貼文功能
Linked Issue
close #545
Details
目前的使用方式，是利用 router 添加 query，如下。
用戶如果要進行搜尋，只需要 enter 或是點左方的 search icon 就可以了。
Possible Impacts
目前 api 串接的部分還未完成，因為目前的 api 好像還沒支持搜尋功能，所以等候端完成後再進行更改。