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

Artemis Console next gen based on hawtIO 4 #5441

Closed
wants to merge 1 commit into from

Conversation

andytaylor
Copy link
Contributor

@andytaylor andytaylor commented Jan 15, 2025

This is a PR to allow developers to build a variant of the latest Artemis Release with the new Artemis Console embedded, for testing. It will not currently merge and isnt intended to. It replaces #5245

The next Artemis Console release candidate, 1.1.0, and this build will use the staged dependency when built.

This PR is a draft and will be updated/reaplced for each Artemis Release until it is decided to merge into main.

@andytaylor andytaylor marked this pull request as draft January 15, 2025 13:46
"favicon": "img/favicon.png"
},
"login": {
"description": "Login page for Artemis Console.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be, "Login page for ActiveMQ Artemis Console."

},
"about": {
"title": "ActiveMQ Artemis Management Console",
"productInfo": [{"name": "Artemis", "value": "${project.version}"}],
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be, "ActiveMQ Artemis".

Comment on lines +123 to +125
<artifactId>apache-artemis-console</artifactId>
<type>zip</type>
<classifier>bin</classifier>
Copy link
Member

Choose a reason for hiding this comment

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

The changes in the pom overall result in inclusion of the console zip and the war in the distribution lib dir.

This should probably marked as provided as its only here for build time use (or maybe it could even be omitted, given it is specified in the dependency unpack execution)

@jbertram
Copy link
Contributor

Replaced by #5467.

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