Skip to content

Conversation

@smnandre
Copy link
Member

@smnandre smnandre commented Dec 6, 2025

Q A
Bug fix? no
New feature? no
Deprecations? no
Documentation? no
Issues Fix #...
License MIT

Changed "platform.sh" name in footer into Upsun

@carsonbot carsonbot added the Status: Needs Review Needs to be reviewed label Dec 6, 2025
hosted with
<span style="font-weight: 700;">
<a href="https://platform.sh"
rel="external noopener noreferrer"
Copy link
Member

Choose a reason for hiding this comment

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

💯

@carsonbot carsonbot added Status: Reviewed Has been reviewed by a maintainer and removed Status: Needs Review Needs to be reviewed labels Dec 6, 2025
@Kocal Kocal added the Site ux.symfony.com label Dec 6, 2025
@carsonbot carsonbot changed the title [minor] fix upson logo & url [Site][minor] fix upson logo & url Dec 6, 2025
@Kocal
Copy link
Member

Kocal commented Dec 8, 2025

Thank you @smnandre.

@Kocal Kocal merged commit 3b24fce into symfony:2.x Dec 8, 2025
4 checks passed
@Kocal Kocal mentioned this pull request Dec 8, 2025
Kocal added a commit that referenced this pull request Dec 8, 2025
This PR was merged into the 2.x branch.

Discussion
----------

[Site] Hide Toolkit menu

| Q              | A
| -------------- | ---
| Bug fix?       | no
| New feature?   | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations?  | no <!-- if yes, also update UPGRADE-*.md and src/**/CHANGELOG.md -->
| Documentation? | no <!-- required for new features, or documentation updates -->
| Issues         | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License        | MIT

So we can release the website with #3220 & #2848, without releasing the Toolkit page.

I didn't have 404 Not Found for `toolkit/*` URLs, I believe hiding the link from the main menu is enough. We have nothing super-important to hide, just the access

Commits
-------

bf53cdc [Site] Hide Toolkit menu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Site ux.symfony.com Status: Reviewed Has been reviewed by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants