Skip to content

Conversation

adriendupuis
Copy link
Contributor

@adriendupuis adriendupuis commented May 30, 2024

Question Answer
JIRA Ticket N/A
Versions master, 4.6
Edition All

Add a hub page and a dedicated category for Ibexa Cloud.

Previews:

Note: I didn't update links to getting_started/install_on_ibexa_cloud/ in new_in_doc.md, I'm counting on the redirects. redirects can't be previewed on https://ez-systems-developer-documentation--2390.com.readthedocs.build/en/2390/resources/new_in_doc/ I note that there is a mix of master and latest in the targets.

Related task ideas:

Checklist

  • Text renders correctly
  • Text has been checked with vale
  • Description metadata is up to date
  • Redirects cover removed/moved pages
  • Code samples are working
  • PHP code samples have been fixed with PHP CS fixer
  • Added link to this PR in relevant JIRA ticket or code PR

@adriendupuis adriendupuis marked this pull request as ready for review June 20, 2024 10:46
@dabrt
Copy link
Contributor

dabrt commented Jun 20, 2024

I like the idea of having Cloud exposed, I remember we've had this discussion. However, do we want to expose Cloud when we have no underlying content in this node, or should we wait until we have the feature guide ready?
@ibexa/documentation?

On an almost unrelated note, while looking ath this page I saw a title "Ibexa Cloud and DDev". The article itself has focus on DDev an I think it should be renamed to "DDev and Ibexa Cloud projects".

@adriendupuis
Copy link
Contributor Author

I like the idea of having Cloud exposed, I remember we've had this discussion. However, do we want to expose Cloud when we have no underlying content in this node, or should we wait until we have the feature guide ready? @ibexa/documentation?

@dabrt
I'am agree that the hub doesn't contain much, and lack a proper introduction. The best introduction would be the product guide. Can a richer and sufficient introduction paragraph be added to this hub in the meantime, or is it impossible to be satisfied until the guide itself is released?

On an almost unrelated note, while looking ath this page I saw a title "Ibexa Cloud and DDev". The article itself has focus on DDev an I think it should be renamed to "DDev and Ibexa Cloud projects".

So true. #2412

Copy link
Contributor

@julitafalcondusza julitafalcondusza left a comment

Choose a reason for hiding this comment

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

There are few things that need to be changed:

  1. All the sections in ToC are expandable (even if there is just one page), it cannot be a single page at this level.
  2. Actually, we have more pages to add in Ibexa Cloud section: "Install on Ibexa Cloud", "DDEV and Ibexa Cloud". Both should be moved to a new section.
  3. Ibexa Cloud product guide will be merged first, we need to remember to add it here.
  4. Links in the tiles on the IC landing page need to be updated due to the 'Install on...' and 'DDEV..." pages being moved to a new section.

@adriendupuis adriendupuis added the Wait with merge PRs that shouldn't be merged instantly label Jul 11, 2024
@adriendupuis adriendupuis removed the Wait with merge PRs that shouldn't be merged instantly label Jul 22, 2024
Copy link
Contributor

@julitafalcondusza julitafalcondusza left a comment

Choose a reason for hiding this comment

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

Please check my comments.

adriendupuis and others added 2 commits July 23, 2024 09:11
Co-authored-by: julitafalcondusza <[email protected]>
Installing on Ibexa Cloud is a hard way to get started.

Co-authored-by: julitafalcondusza <[email protected]>
@adriendupuis adriendupuis requested a review from mnocon July 23, 2024 07:47
Copy link
Contributor

@mnocon mnocon left a comment

Choose a reason for hiding this comment

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

Thanks!

@adriendupuis adriendupuis merged commit a8a602d into master Jul 24, 2024
@adriendupuis adriendupuis deleted the ibexa-cloud-hub branch July 24, 2024 06:50
adriendupuis added a commit that referenced this pull request Jul 24, 2024
---------

Co-authored-by: julitafalcondusza <[email protected]>
(cherry picked from commit a8a602d)
adriendupuis added a commit that referenced this pull request Jul 30, 2024
---------

Co-authored-by: julitafalcondusza <[email protected]>
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.

4 participants