Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

WIP: add push mqtt5 props card #782

Merged
merged 1 commit into from
Nov 22, 2021
Merged

Conversation

oceanlvr
Copy link
Contributor

@oceanlvr oceanlvr commented Nov 15, 2021

PR Checklist

If you have any questions, you can refer to the Contributing Guide

What is the current behavior?

Please describe the current behavior and link to a relevant issue.

Issue Number

Example: #123

What is the new behavior?

Please describe the new behavior or provide screenshots.

Does this PR introduce a breaking change?

  • Yes
  • No

Specific Instructions

Are there any specific instructions or things that should be known prior to review?

Other information

@oceanlvr oceanlvr requested a review from ysfscream November 15, 2021 10:02
@oceanlvr oceanlvr marked this pull request as draft November 15, 2021 10:02
@oceanlvr
Copy link
Contributor Author

image
@ysfscream

@ysfscream
Copy link
Member

@oceanlvr Good Job! But sorry, we'll use the first style.

image

@oceanlvr
Copy link
Contributor Author

image

So sorry for my delay commit. @ysfscream

@oceanlvr oceanlvr marked this pull request as ready for review November 21, 2021 07:01
@oceanlvr
Copy link
Contributor Author

@ysfscream
Copy link
Member

@oceanlvr What properties are not received? I test user properties is normal, I see your issue using MQTT broker, not EMQX, is your MQTT broker not support v5.0?

Comment on lines +80 to +81
<el-button size="mini" type="outline" @click="submitForm">save</el-button>
<el-button size="mini" type="outline" @click="resetForm">reset</el-button>
Copy link
Member

Choose a reason for hiding this comment

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

Translate save and reset.

@ysfscream ysfscream merged commit 739da1e into emqx:master Nov 22, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants