goodjoblife / GoodJobShare

GoodJob 職場透明化運動
https://www.goodjob.life/
MIT License
82 stars 11 forks source link

[經驗單篇] 修正sections #1362

Closed peteranny closed 4 months ago

peteranny commented 4 months ago

這個 PR 是?

https://api-dev.goodjob.life/graphql

由於 experience API 將 sections 從 Experience 移出,變成 WorkExperience 和 InterviewExperience 有各自的 sections,寫 graphql 時要分開指定

由於 WorkExperience.sections.subtitle 和 InterviewExperience.sections.subtitle 有不同的型別,所以在打 api 時先取不同名稱,抓回來後再統一命名為 subtitle 以作最簡單的向前相容

我應該如何手動測試?

mark86092 commented 4 months ago

對耶,還有 __typename 可以用 :+1:

mark86092 commented 4 months ago

給定一個不存在的 experience eg: https://www-dev.goodjob.life/experiences/657736471413e424a5a37b01

會導致

image

這個 bug 也影響 master branch (eg: https://www-stage.goodjob.life/experiences/66571c10a59b5c8766cbd061