Skip to content

Conversation

@ansd
Copy link
Member

@ansd ansd commented Apr 5, 2024

This is the documentation for rabbitmq/rabbitmq-server#10873

@cloudflare-workers-and-pages
Copy link

Deploying rabbitmq-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: bed3e5e
Status: ✅  Deploy successful!
Preview URL: https://e7b23521.rabbitmq-website.pages.dev
Branch Preview URL: https://amqp.rabbitmq-website.pages.dev

View logs

@michaelklishin michaelklishin changed the title Add AMQP address v2 4.x: add AMQP address v2 Apr 5, 2024
@dumbbell
Copy link
Collaborator

dumbbell commented Apr 9, 2024

We will be able to review and merge this patch after #1878 which creates the 3.13 docs folder.

@dumbbell
Copy link
Collaborator

The 3.13 docs were branched (#1878). Therefore, this pull request can be rebased, reviewed and merged.

@ansd ansd merged commit 876868d into main Apr 15, 2024
@ansd ansd deleted the amqp branch April 15, 2024 12:29
@dumbbell
Copy link
Collaborator

@ansd: I don’t know if it was intentional, but the page added by this pull request is not linked by another page or a sidebar. Therefore it can only be reached by search or by typing the URL. Is it on purpose?

@ansd
Copy link
Member Author

ansd commented Apr 15, 2024

@dumbbell I would like to add the new page to the sidebar. My hope was that this gets done automatically. It looks like I have to manually add the new page to https://github.com/rabbitmq/rabbitmq-website/blob/main/sidebarsDocs.js?

@dumbbell
Copy link
Collaborator

Indeed, the ToC is edited manually to get the order, hierarchy and labels right.

sidebarDocs.js is the correct file to edit. It is divided to target two types of users:

  • Developers under "How to Use RabbitMQ"
  • Admins under "How to Manage RabbitMQ" (and "How to Monitor RabbitMQ")

ansd added a commit that referenced this pull request Apr 15, 2024
@ansd
Copy link
Member Author

ansd commented Apr 15, 2024

Thanks for spotting this. I added the new page to the sidebar: 1942ef9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants