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

docs: update download-and-launch.adoc #2747

Closed
wants to merge 5 commits into from

Conversation

marcobonita
Copy link
Contributor

Adding a few details: prerequisites and how to update files

Copy link
Contributor

github-actions bot commented Jun 26, 2024

📝 Check the pages that have been modified

In order to merge this pull request, you need to check your updates with the following url.

🔍 Updated pages

The following pages were updated, please ensure that the display is correct:

Copy link
Contributor

github-actions bot commented Jun 27, 2024

🎊 PR Preview 6572e2f has been successfully built and deployed to https://bonitasoft-bonita-doc-deploy-pr-2747.surge.sh

🕐 Build time: 0.01s

🤖 By surge-preview

@@ -6,6 +6,11 @@
For Subscription editions only.
====

== Pre-requisites
* Docker installed
* Nginx installed
Copy link
Member

Choose a reason for hiding this comment

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

Isn't nginx setup using a Docker image ?

@@ -175,7 +180,7 @@ Run the following command to launch Bonita UI Builder:
docker compose up -d
----

Then, you can access Bonita UI Builder at `http://localhost`.
Then, you can access Bonita UI Builder at `http://localhost:8090`.
Copy link
Member

Choose a reason for hiding this comment

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

In the demo, the reverse proxy was handling the redirection to the proper port. Did it changed ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

When I tested it, it didn't redirect me properly to the right URL, hence this change

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK...it just worked now 🤔

Copy link
Member

@rbioteau rbioteau left a comment

Choose a reason for hiding this comment

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

In the Download and launch Bonita UI Builder page.
The docker-compose file should use an antora variable to store the bonita-ui-builder image version.

@marcobonita marcobonita closed this Jul 1, 2024
@marcobonita
Copy link
Contributor Author

Some of the proposed changes are not relevant + kind of redundant with #2739

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

Successfully merging this pull request may close these issues.

2 participants