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

[經驗單篇] 允許觀看自己的文章 #1438

Merged
merged 22 commits into from
Oct 15, 2024
Merged

[經驗單篇] 允許觀看自己的文章 #1438

merged 22 commits into from
Oct 15, 2024

Conversation

peteranny
Copy link
Contributor

@peteranny peteranny commented Sep 7, 2024

Close #1392

這個 PR 是?

新增 MyPublishes API 拿取自己的 publish IDs,存進 redux。

新增 useIsMyPublishId 簡化每個需要檢查是否是自己的經驗/薪時的頁面。

Screenshots

薪時 經驗
截圖 2024-09-20 凌晨1 21 20 截圖 2024-09-20 凌晨1 21 33

我應該如何手動測試?

確認自己登入且沒有權限

  • 到自己留的經驗,應可見。
  • 到非自己留的經驗,應不可見。
  • 到自己留的公司頁面,應只有自己的經驗/薪時可見。
  • 到自己留的職稱頁面,應只有自己的經驗/薪時可見。

確認自己未登入

  • 到自己留的經驗/薪時,應不可見。
  • 登入後,應可見自己留的資料。

This reverts commit d9272e5af1fc5c252fd0c5808e9bff1a294fc122.
@peteranny peteranny requested a review from a team September 19, 2024 17:26
@peteranny peteranny marked this pull request as ready for review September 19, 2024 17:27
@peteranny peteranny requested a review from mark86092 September 19, 2024 17:42
@barry800414
Copy link
Contributor

確認自己登入且沒有權限

  • 到自己留的經驗,應可見。
  • 到非自己留的經驗,應不可見。
  • 到自己留的公司頁面,應只有自己的經驗/薪時可見。
  • 到自己留的職稱頁面,應只有自己的經驗/薪時可見。

確認自己未登入

  • 到自己留的經驗/薪時,應不可見。
  • 登入後,應可見自己留的資料。

Copy link
Contributor

@barry800414 barry800414 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the fix.

@peteranny peteranny requested a review from mark86092 September 21, 2024 17:30
@peteranny
Copy link
Contributor Author

@barry800414 @mark86092 剛想到 isMyPublishId 若直接與 canView 合併成一個函數,語意上會更清楚,所以又多做一點 refactor,再麻煩看一下 🙏🏼

@barry800414
Copy link
Contributor

barry800414 commented Sep 25, 2024

確認自己登入且沒有權限

  • 到自己留的經驗,應可見。
  • 到非自己留的經驗,應不可見。
  • 到自己留的公司頁面,應只有自己的經驗/薪時可見。
  • 到自己留的職稱頁面,應只有自己的經驗/薪時可見。

確認自己未登入

  • 到自己留的經驗/薪時,應不可見。
  • 登入後,應可見自己留的資料 ----> @peteranny 登入後要重新整理頁面,才會可以看到自己留的資料

@peteranny
Copy link
Contributor Author

* [ ]  登入後,應可見自己留的資料 ----> @peteranny 登入後要重新整理頁面,才會可以看到自己留的資料

Nice catch! Fixed :-D Please verify.

@mark86092
Copy link
Contributor

如果是要確保 user 登入狀態改變會觸發 queryMyPublishId

可以參考:
https://github.com/goodjoblife/GoodJobShare/blob/master/src/components/App/Header/index.js#L70-L74
的做法

@peteranny
Copy link
Contributor Author

已更新成用 isLoggedIn 觸發更新,移除 token dependency

@peteranny peteranny merged commit 0884ebd into master Oct 15, 2024
8 checks passed
@peteranny peteranny deleted the fix/show-my-exp branch October 15, 2024 17:02
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.

自己填寫的資料自己不能看
3 participants