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

feat: solace add per operation fields #237

Merged
merged 3 commits into from
Jan 30, 2024

Conversation

GreenRover
Copy link
Collaborator

Description

  • add new field: timeToLive on a per operation bases.
    To enable to documentate what the a good valid related to the business logic of this event. For example if a temperature reading is publish every 30 sec. A ttl >30sec would make no sense.
    I choose a per operation bases to align to https://github.com/asyncapi/bindings/blob/master/mqtt/README.md
  • add new field: priority on a per operation bases.
    To enable to documentate the importent ratings of events.
  • add new field: dmqEligible on a per operation bases.
    To enable a message consumer to provide all necessary Information how the expected messages should look like.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide 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.

Copy link
Member

@CameronRushton CameronRushton left a comment

Choose a reason for hiding this comment

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

Thanks for this. A couple simple requests to address & thank you for volunteering to update spec-json-schemas as well.

solace/README.md Outdated
`destinations`|List of Destination Objects|Destination Objects are described next.
Field Name | Type | Description
---|---------------------------------------------------------------------------------|---
`bindingVersion`| String |The current version is 0.3.0
Copy link
Member

Choose a reason for hiding this comment

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

Please update the binding version to 0.4.0 throughout the document.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

changed

solace/README.md Outdated
Comment on lines 40 to 41
`timeToLive` | Integer \| [Schema Object][schemaObject] \| [Reference Object](referenceObject) | Interval in milliseconds or a *Schema Object* containing the definition of the lifetime of the message.
`priority` | Integer \| [Schema Object][schemaObject] \| [Reference Object](referenceObject) | The valid priority value range is 0-255 with 0 as the lowest priority and 255 as the highest or a *Schema Object* containing the definition of the priority.
Copy link
Member

Choose a reason for hiding this comment

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

[Schema Object][schemaObject] | Reference Object

This appears to be broken. I think for v3, it's supposed to go here for referenceObject (https://www.asyncapi.com/docs/reference/specification/v3.0.0#referenceObject). Similarly for schemaObject.

Copy link
Collaborator Author

@GreenRover GreenRover Jan 30, 2024

Choose a reason for hiding this comment

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

thanks for pointing out. i missed to links. Added it at the bottom.

@GreenRover
Copy link
Collaborator Author

I changed example to async api 3.0 in same pr to avoid a version incretion to 0.5.0 for that

Copy link
Member

@CameronRushton CameronRushton left a comment

Choose a reason for hiding this comment

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

Looks great. We really appreciate your help!

@GreenRover
Copy link
Collaborator Author

/rtm

@asyncapi-bot asyncapi-bot merged commit 7093d99 into asyncapi:master Jan 30, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants