Fix bug of JSON parse

[HaoQiuji-Pavel]

No description provided.

[HaoQiuji-Pavel]

@nighca 寒星老师，这个pr修改了上次CodingStep的意见。然后改消费方代码是因为万欣后端传过来的是json字符串，前端需要JSON.parse才能使用

[nighca]

改消费方代码是因为万欣后端传过来的是json字符串，前端需要JSON.parse才能使用

要看这个 parse 在哪里做。如果我们约定的 backend 模块返回的 step 就是 object 本身而不是对应的 JSON 字符串，那么应当由 backend 模块自己做 parse。可能是在 backend 的 go 代码中，最差也应该在 apis/guidance.ts 中，而不应该是具体消费 step 数据的业务代码中

[qiniu-x]

[Git-flow] Hi @HaoQiuji-Pavel, There are some suggestions for your information:

---

Rebase suggestions

• Following commits seems generated via git merge

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

• Following commits have duplicated messages

  feat: Add StoryLine

  feat: Add StoryLine

  feat: Add StoryLine

  feat: Add StoryLine

  feat: Add StoryLine

  feat: Add StoryLine

  feat: Add StoryLine

  feat: Add VideoPlayer reset

  feat: Add VideoPlayer reset

  feat: Add CodingStep

  feat: Add CodingStep

  feat: Add CodingStep

  feat: Add CodingStep

  feat: add NodeTaskPlayer

  feat: add NodeTaskPlayer

  fix: modify client.ts

  fix: modify client.ts

  fix: modify client.ts

  feat: frontend and backend integration test

  feat: frontend and backend integration test

  feat: frontend and backend integration test

  feat: frontend and backend integration test

  fix: fix bug of JSON parse

  fix: fix bug of JSON parse

  fix: fix bug of JSON parse

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

  Merge branch 'trailhead_guidance' of https://github.com/goplus/builder into guidance_develop

Which seems insignificant, recommend to use git rebase command to reorganize your PR.

For other git-flow instructions, recommend refer to these examples.

If you have any questions about this comment, feel free to raise an issue here:

• https://github.com/qiniu/reviewbot

[qiniu-ci]

This PR has been deployed to the preview environment. You can explore it using the preview URL.

Warning

Please note that deployments in the preview environment are temporary and will be automatically cleaned up after a certain period. Make sure to explore it before it is removed. For any questions, contact the Go+ Builder team.

[HaoQiuji-Pavel]

@nighca

要看这个 parse 在哪里做。如果我们约定的 backend 模块返回的 step 就是 object 本身而不是对应的 JSON 字符串，那么应当由 backend 模块自己做 parse。可能是在 backend 的 go 代码中，最差也应该在 apis/guidance.ts 中，而不应该是具体消费 step 数据的业务代码中

明白，已修改