Skip to content

IBX-8172: OpenAPI compatible location copy endpoint #110

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 47 commits into from
Jul 26, 2024
Merged

IBX-8172: OpenAPI compatible location copy endpoint #110

merged 47 commits into from
Jul 26, 2024

Conversation

@barw4
Copy link
Contributor

@barw4 barw4 commented Jun 24, 2024 •
edited
Loading

🎫 Issue IBX-8172

Related PRs:

#107

Description:

The new endpoint follows the given specification:

POST /api/ibexa/v2/content/locations/1/2/71
Content-Type: application/vnd.ibexa.api.CopyLocationInput+json
Accept: application/json
{
    "CopyLocationInput": {
        "destination": "/1/5"
    }
}

Documentation:

TBD

@barw4 barw4 added Feature New feature request Ready for review labels Jun 24, 2024
@barw4 barw4 self-assigned this Jun 24, 2024
@barw4 barw4 requested a review from a team June 24, 2024 16:37
@barw4 barw4 force-pushed the ibx-8180-swap-location-endpoint-OpenAPI branch from 3268d62 to 55e48bf Compare June 27, 2024 16:25
@barw4 barw4 force-pushed the ibx-8172-copy-location-endpoint-OpenAPI branch 2 times, most recently from 2fe9824 to dd3de8e Compare June 28, 2024 08:51
@barw4 barw4 force-pushed the ibx-8180-swap-location-endpoint-OpenAPI branch from 62e73b9 to 8abbae4 Compare June 28, 2024 14:40
@barw4 barw4 force-pushed the ibx-8172-copy-location-endpoint-OpenAPI branch from 77b9b06 to 94cbbb0 Compare June 28, 2024 14:55
@barw4 barw4 force-pushed the ibx-8180-swap-location-endpoint-OpenAPI branch from 6969142 to 4297243 Compare July 2, 2024 22:21
@barw4 barw4 force-pushed the ibx-8172-copy-location-endpoint-OpenAPI branch from d914b7e to e95af2a Compare July 3, 2024 08:24
konradoboza
konradoboza approved these changes Jul 3, 2024
View reviewed changes
@konradoboza konradoboza requested a review from a team July 3, 2024 10:49
@barw4 barw4 force-pushed the ibx-8180-swap-location-endpoint-OpenAPI branch from 0227bcc to 989b2f1 Compare July 16, 2024 07:46
Base automatically changed from ibx-8180-swap-location-endpoint-OpenAPI to main July 17, 2024 09:11
@katarzynazawada katarzynazawada self-assigned this Jul 24, 2024
barw4 added 20 commits July 24, 2024 19:53
@barw4
IBX-8176: Implemented routing language expression extension handling …
8aa0b28 
…`Content-Type` header
@barw4
IBX-8176: Added unit tests
050b8d3
@barw4
IBX-8176: Applied review remarks
1a2dcd0
@barw4
IBX-8176: Refactored solution
4e6a696
@barw4
IBX-8176: Refactoring
86951ef
@barw4
IBX-8176: Rename the expression function
0baa83b
@barw4
IBX-8180: Added integration test
5c8cfd1
@barw4
IBX-8180: Fixed routing
ae16557
@barw4
IBX-8176: Implemented routing language expression extension handling …
2fc7b69 
…`Content-Type` header
@barw4
IBX-8176: Added unit tests
d977b9a
@barw4
IBX-8176: Applied review remarks
591af40
@barw4
IBX-8176: Refactored solution
e303b2b
@barw4
IBX-8176: Applied review remarks
6c56e4e
@barw4
IBX-8176: Refactoring
8d4e952
@barw4
IBX-8176: Rename the expression function
c29e384
@barw4
IBX-8172: OpenAPI compatible location copy endpoint
5b3b0f2
@barw4
IBX-8172: Update functional tests
b3cfd28
@barw4
IBX-8172: Fixed routing
318b321
@barw4
IBX-8172: Fixed routing
d3a627d
@barw4
IBX-8172: Applied review remark
85ea622
@barw4 barw4 force-pushed the ibx-8172-copy-location-endpoint-OpenAPI branch from 74a1f4e to 85ea622 Compare July 24, 2024 17:54
@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@Steveb-p Steveb-p merged commit c4b854a into main Jul 26, 2024
@Steveb-p Steveb-p deleted the ibx-8172-copy-location-endpoint-OpenAPI branch July 26, 2024 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Steveb-p Steveb-p Steveb-p approved these changes

@ciastektk ciastektk ciastektk approved these changes

@ViniTou ViniTou ViniTou approved these changes

@konradoboza konradoboza konradoboza approved these changes

@katarzynazawada katarzynazawada katarzynazawada approved these changes

Assignees

@Steveb-p Steveb-p

@alongosz alongosz

@tischsoic tischsoic

@barw4 barw4

@katarzynazawada katarzynazawada

Labels

Feature New feature request QA approved

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@barw4 @Steveb-p @ciastektk @ViniTou @konradoboza @katarzynazawada @alongosz @tischsoic