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-2987 improve message body display #3350

Closed
wants to merge 1 commit into from

Conversation

jbertram
Copy link
Contributor

No description provided.

<h2>Headers</h2>
<h4>Message ID: {{$ctrl.currentMessage.messageID}}</h4>

<h4>Displaying body as <span ng-bind="$ctrl.currentMessage.textMode"></span></h4>
Copy link
Contributor

Choose a reason for hiding this comment

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

did you test this on both AMQP and Core?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did a quick test for this with core, but this change is really just superficial. There's no semantic change here. The same information is being displayed as always, just in a slightly different way.

Copy link
Contributor

Choose a reason for hiding this comment

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

I see. I haven't fully read the whole PR at the time I posted this.. I thought you implemented some visutalzation of the body, in which case you would have needed to change some java code..

it's fine then.. thanks

@asfgit asfgit closed this in 8259441 Nov 19, 2020
@jbertram jbertram deleted the ARTEMIS-2987 branch June 10, 2021 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants