Hide Message ID block if messageId is not set

[Vaniog]

Description

Remove block [Message ID] presented on the photo, if message has no property "messageId" (as in attached specification)

Reasons

This duplicate information and take a lot of space

Attachments

asyncapi: 3.0.0
info:
  title: Hello world application
  version: '0.1.0'
channels:
  hello:
    address: 'hello'
    messages:
      sayHelloMessage:
        payload:
          type: string
operations:
  receiveHello:
    action: 'receive'
    channel:
      $ref: '#/channels/hello'

[github-actions]

Welcome to AsyncAPI. Thanks a lot for reporting your first issue. Please check out our contributors guide and the instructions about a basic recommended setup useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

[ItshMoh]

Hey @Vaniog can you explain what are you trying to say in the issue. As the spec you have shared is containing the {messagId}
. For this doc it is 'sayHelloMessage' . You can refer here.
let me know what are your thoughts.

[Vaniog]

In this case messageId is same as message name. But you can set messageId property, as in spec below.

components:
  messages:
    sayHelloMessage:
      messageId: notSayHelloMessage
      payload:
        type: object
        properties:
          name:
            type: string

So message name and messageId will be different
In case, which i provided in issue, these properties are same, so i dont want to see both of them in message header (screenshot)

[ItshMoh]

Hey @Vaniog In asyncapi version 3.0.0 it is not valid. You can't specify the messageID in the spec. Here You can again refer.
I have tried you spec in studio.

[Vaniog]

Yes. And so displaying message name and messageId separately in viewer is deprecated too.

[Vaniog]

But you can see duplicate ids here https://github.com/asyncapi/asyncapi-react/blob/master/library/src/containers/Messages/Message.tsx#L62, and here https://github.com/asyncapi/asyncapi-react/blob/master/library/src/containers/Messages/Message.tsx#L97

[ItshMoh]

Hey @AceTheCreator Would it be fine to remove one of the messageID, as both are the same. Is it for some other purposes?
what's your suggestion, if it is a valid issue or not?

[AceTheCreator]

@ItshMoh, yea, we should remove the first one 👍🏾

[asyncapi-bot]

🎉 This issue has been resolved in version 2.3.3 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀