Skip to content

Conversation

@adriendupuis
Copy link
Contributor

@adriendupuis adriendupuis commented Jun 8, 2022

Question Answer
JIRA Ticket IBX-1430
Versions 4.4

Update/Complete REST API reference

  • Add missing methods to existing routes
  • Add missing routes
  • Update namespace (xmlns) from ez.no to ibexa.co (kept for another latter PR)
  • Harmonize deprecation and removal flags
  • Add a test command to compare the built reference against real routing configuration (moved to IBX-6107: REST API ref tester #1901)
  • Fix build command's templates to quit skipping some methods

Related and dependencies:

Changelog:

Not treated and kept for an other latter PR:

  • /content/binary/
  • /languages/
  • /segments/ and /segment_groups/
  • /personalization/v1/
  • /views/
  • Files renaming to remove "ez-" prefix

description: OK - The basket has been deleted
400:
description: Error - The basket does not exist
403:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@adriendupuis
Copy link
Contributor Author

I'll now target 4.4

I see a lot of depreciation in ez-commerce.raml 😓 😛

@adriendupuis adriendupuis changed the title IBX-1430: Complete REST API reference IBX-1430: Complete 4.4 REST API reference Jan 19, 2023
@adriendupuis adriendupuis marked this pull request as ready for review January 24, 2023 13:45
@adriendupuis adriendupuis requested a review from Steveb-p January 24, 2023 13:45
@adriendupuis
Copy link
Contributor Author

I split this REST API ref. update. I'll continue in another one.

Here are added 32 pairs (method+route), few existing ones are fixed, that's a lot to review.

There is still a lot to improve (examples addition, being sure that every possible responses are covered, enhance the test command, etc.)

@DominikaK DominikaK force-pushed the master branch 3 times, most recently from 33f6b7e to 2b50cbd Compare February 2, 2023 14:26
adriendupuis and others added 13 commits February 17, 2023 09:41
/content/objects: ez-content-object.raml → ez-content-objects.raml
/content/locations: ez-content-location.raml → ez-content-location.raml
/content/sections: ez-content-section.raml → ez-content-sections.raml
- GET /content/objects/{contentId}/versions/{versionNo}/schedule
- PUT /content/objects/{contentId}/versions/{versionNo}/schedule/{publicationTimestamp}
- PATCH /content/objects/{contentId}/versions/{versionNo}/schedule/{publicationTimestamp}
Prefixed /api/datebasedpublisher/v1 instead of /api/ibexa/v2

- GET /content/objects/{contentId}/versions/{versionNo}/schedule
- PUT /content/objects/{contentId}/versions/{versionNo}/schedule/{publicationTimestamp}
- PATCH /content/objects/{contentId}/versions/{versionNo}/schedule/{publicationTimestamp}
When a route had more than one sub-routes, its methods were skipped at HTML generation time.

```
/tmp:
    displayName: 1st level route
    get:
        displayName: Immediate GET
    /first:
        get:
            displayName: Test a URI element
```
was documenting
- GET /tmp
- GET /tmp/first

while
```
/tmp:
    displayName: 1st level route
    get:
        displayName: Immediate GET
    /first:
        get:
            displayName: Test a URI element
    /second:
        post:
            displayName: Test another URI element
```
was only documenting
- GET /tmp/first
- GET /tmp/second
and was skipping GET /tmp
@adriendupuis adriendupuis mentioned this pull request Feb 17, 2023
7 tasks
@adriendupuis adriendupuis merged commit c4bd49e into master Mar 2, 2023
@adriendupuis adriendupuis deleted the update-rest-api-ref branch March 2, 2023 13:32
adriendupuis added a commit that referenced this pull request Mar 2, 2023
@adriendupuis
Copy link
Contributor Author

After being merged into master branch, has been cherry-picked in 4.4 branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants