Skip to content
Snippets Groups Projects

Depth parameter

Merged Ghost User requested to merge depth-parameter into master

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Author Contributor

    @sylvain @clement I pushed the branch into stagin for test purpose. If it's ok for you, please merge.

    1. test that depth is 1 : https://test-paris.happy-dev.fr/job-offers/
    2. Test that the app still working
    3. test that the perf are better

    By Jean-Baptiste on 2019-02-04T10:41:21 (imported from GitLab project)

  • JB, I think the right way to test would be: generate an API with depth 2, validate on the json-ld playground that it breaks the browser, then change the depth to 1 and validate that it's fixed

  • Ghost User added 12 commits

    added 12 commits

    Compare with previous version

    By Jean-Baptiste on 2019-02-05T09:00:32 (imported from GitLab project)

  • merged

    By Jean-Baptiste on 2019-02-05T09:01:25 (imported from GitLab project)

  • Ghost User mentioned in commit 333b5803

    mentioned in commit 333b5803

    By Jean-Baptiste on 2019-02-05T09:01:25 (imported from GitLab project)

  • Author Contributor

    @clement I've tested on the json-ld playground but it's still broken. Maybe I don't test the right way on json ld playgroung. Could we redo it together ?

    By Jean-Baptiste on 2019-02-05T09:10:03 (imported from GitLab project)

  • Ouch. I guess the best thing to do then is to open an issue on jsonld.js.

    There is a call with the solid community tonight. Maybe if you can be there you can raise the issue. I think the author of rdflib is going to be present.

Please register or sign in to reply
Loading