Skip to content
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.

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

IBX-4796: Proper redirect after content publish #64

Merged
merged 4 commits into from
Feb 6, 2023

Conversation

barw4
Copy link
Member

@barw4 barw4 commented Jan 18, 2023

JIRA: https://issues.ibexa.co/browse/IBX-4796

Publishing should redirect to a referrer Location now.

@barw4 barw4 added Bug Something isn't working Ready for review labels Jan 18, 2023
@barw4 barw4 requested a review from a team January 18, 2023 10:04
@barw4 barw4 self-assigned this Jan 18, 2023
@konradoboza konradoboza requested a review from a team January 30, 2023 08:53
@konradoboza konradoboza requested a review from a team January 30, 2023 11:05
@webhdx webhdx requested a review from a team February 1, 2023 07:36
@ciastektk ciastektk requested a review from a team February 1, 2023 07:50
@ViniTou ViniTou self-requested a review February 1, 2023 08:44
Copy link
Contributor

@ViniTou ViniTou left a comment

Choose a reason for hiding this comment

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

It seems, that failing browser test is relevant, could you try to reproduce it in UI?

Scenario: Create a folder without a draft                 # vendor/ezsystems/ezplatform-content-forms/features/ContentEdit/create_without_draft.feature:6
    Given that I have permission to create folders          # EzSystems\EzPlatformContentForms\Behat\Context\ContentEditContext::thatIHavePermissionToCreateFolders()
    And there is a Content Type "folder" with the id "1"    # EzSystems\EzPlatformContentForms\Behat\Context\ContentTypeContext::thereIsAContentTypeWithId()
    When I go to "content/create/nodraft/folder/eng-GB/2"   # EzSystems\EzPlatformContentForms\Behat\Context\ContentEditContext::visit()
    Then I should see a folder content edit form            # EzSystems\EzPlatformContentForms\Behat\Context\ContentEditContext::iShouldSeeAContentEditForm()
    When I fill in the folder edit form                     # EzSystems\EzPlatformContentForms\Behat\Context\ContentEditContext::iFillInTheFolderEditForm()
    And I press "Publish"                                   # EzSystems\EzPlatformContentForms\Behat\Context\ContentEditContext::pressButton()
    Then I am on the View of the Content that was published # EzSystems\EzPlatformContentForms\Behat\Context\ContentEditContext::iAmOnTheViewOfTheContentThatWasPublished()
      Behat\Mink\Exception\ElementNotFoundException: Element matching css "span.ezstring-field" not found. in vendor/friends-of-behat/mink/src/WebAssert.php:418
      ```

@ViniTou ViniTou self-requested a review February 1, 2023 08:46
@sonarcloud
Copy link

sonarcloud bot commented Feb 1, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@barw4
Copy link
Member Author

barw4 commented Feb 1, 2023

It seems, that failing browser test is relevant, could you try to reproduce it in UI?

@ViniTou Indeed, a parent Location was resolved as a referrerLocation for nodraft actions, fixed in 143a0d0.

@bogusez bogusez self-assigned this Feb 6, 2023
@webhdx webhdx merged commit 69f9088 into 1.3 Feb 6, 2023
@webhdx webhdx deleted the ibx-4796-redirect-after-publish branch February 6, 2023 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
7 participants