Skip to content

Added deprecations page for Cohesivo 6.0#3279

Open
mnocon wants to merge 11 commits into
5.0from
deprecations-page
Open

Added deprecations page for Cohesivo 6.0#3279
mnocon wants to merge 11 commits into
5.0from
deprecations-page

Conversation

@mnocon

@mnocon mnocon commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Target: 4.6, 5.0, 6.0

Adding deprecations page for the upcoming Cohesivo 6.0, to let Partners know about the upcoming changes ahead of time.

For now, the following changes are covered:

As a new Rector rule is added, there's a need for dedicated Rector page:

I've extracted the exsiting content (from update_to_5.0.md and package's README) into a separate page we can refer people to.

As this might be the first time people hear about Cohesivo, I've added 2 links explaining the business reasoning.

Status: this needs to be merged with the upcoming release, as this is when the Rector rule becomes available.

Link checker results should be ignored, there's a known issue here (fixed in #3280 )

Companion PRs:

TODO BEFORE MERGING/BEFORE RELEASE

MAKE SURE da9f87a IS UP TO DATE (IMPLEMENTED AND MERGED) BEFORE THE RELEASE (https://ibexa.atlassian.net/browse/IBX-12084)

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

Preview of modified files

Preview of modified Markdown:

@mnocon mnocon changed the base branch from 5.0 to messenger-5.0 July 7, 2026 07:14
@mnocon mnocon force-pushed the deprecations-page branch from afe1bed to a14a990 Compare July 7, 2026 07:17
@mnocon mnocon added the Wait with merge PRs that shouldn't be merged instantly label Jul 7, 2026
@mnocon mnocon force-pushed the deprecations-page branch from a14a990 to a994244 Compare July 7, 2026 07:26
@mnocon mnocon marked this pull request as ready for review July 7, 2026 07:27
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 763594
🔗 Unique 15085
✅ Successful 7290
⏳ Timeouts 0
🔀 Redirected 1
👻 Excluded 756301
❓ Unknown 0
🚫 Errors 3
⛔ Unsupported 0

Errors per input

Errors in site/release_notes/cohesivo_v6.0_deprecations/index.html

Errors in site/resources/rector/index.html

Redirects per input

Redirects in site/release_notes/cohesivo_v6.0_deprecations/index.html

Full Github Actions output

@mnocon mnocon force-pushed the deprecations-page branch from a994244 to 5e09d90 Compare July 7, 2026 12:57
Base automatically changed from messenger-5.0 to 5.0 July 10, 2026 14:26
@mnocon mnocon force-pushed the deprecations-page branch from 5e09d90 to 7e4904b Compare July 13, 2026 08:47
@mnocon mnocon force-pushed the deprecations-page branch from 7e4904b to 91b5768 Compare July 13, 2026 08:48
@mnocon mnocon requested a review from konradoboza July 13, 2026 12:53

@bnowak bnowak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Shouldn't it be merged into the lowest version (4.6) and be merged-up to all higher versions? I don't know what rules do we have regarding documentation versioning?


| Deprecated since | Entity | Change |
| --- |------------------------------------------------------------------------------------|-----------------------------------------------------------|
| v5.0.9 | [`SudoStamp`](background_tasks.md#sudostamp) | No longer attached automatically to every dispatched message. For messages that should be processed without taking permissions into account, always attach the SudoStamp manually. |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All other Entity table entries use Fully Qualified Class Names, this one is not. For consistency we could use its long version as well: \Ibexa\Contracts\Messenger\Stamp\SudoStamp.

Comment thread docs/resources/rector.md Outdated
Add the Composer dependency:

``` bash
composer require --dev ibexa/rector:[[= latest_tag_5_0 =]]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I wouldn't point it to any specific version, as that would lock/block any potential future updates of that package. If we leave only composer require --dev ibexa/rector - composer will install its latest available matching version. This way, the client will be able to update that package too with regular composer update 😉

@konradoboza konradoboza left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! Do we have a plan to make some more noise about this page to create a habit it should be checked from time to time for the new additions?

Comment thread docs/resources/rector.md Outdated
vendor/bin/rector
```

## Refactor JavaScipt code

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think it should be 'JavaScript' ;)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 787c6b8 , thanks 🙈

@mnocon

mnocon commented Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

Shouldn't it be merged into the lowest version (4.6) and be merged-up to all higher versions? I don't know what rules do we have regarding documentation versioning?

For versioning we don't use a flow based on git merges, instead we squash the commits into a single one when merging and then cherry-pick it manully to the appropriate versions. I know it's a bit unusual, but it works 😉

Suggestions applied in 1fda0e8, thank you!

Looks good! Do we have a plan to make some more noise about this page to create a habit it should be checked from time to time for the new additions?

Yes, the plan is to mention this page in every release notes when we add a new deprecation to this list, to raise awareness around it.

@mnocon mnocon requested a review from a team July 13, 2026 13:35
@ibexa-workflow-automation-1 ibexa-workflow-automation-1 Bot requested review from adriendupuis, dabrt and julitafalcondusza and removed request for a team July 13, 2026 13:35

As announced during Ibexa Summit 2026, [Ibexa DXP will be renamed to Cohesivo](https://www.ibexa.co/blog/redefining-the-dxp-from-execution-to-orchestration) to support the new [orchestration platform approach](https://www.ibexa.co/blog/the-orchestration-era).

To learn more about the new brand, visit the [Cohesivo official site](https://cohesivo.com).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Discussed the sentences above with Kamil - it's ok, and improvements to the cohesivo.com site will be released on 16.06 (2 days before the next planned release when I'd like to merge this)

@julitafalcondusza julitafalcondusza left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please check my comments.

Comment thread docs/resources/rector.md Outdated
Comment thread docs/resources/rector.md Outdated
Comment thread docs/resources/rector.md Outdated
Comment thread docs/resources/rector.md Outdated
Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com>
@mnocon mnocon requested a review from julitafalcondusza July 15, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs DOC review Wait with merge PRs that shouldn't be merged instantly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants