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: bump avsc and i18n example files #126

Merged
merged 7 commits into from
May 22, 2024
Merged

Conversation

JimMoen
Copy link
Member

@JimMoen JimMoen commented May 6, 2024

No description provided.

@JimMoen
Copy link
Member Author

JimMoen commented May 6, 2024

CI failed temporarily, re-run failed check after emqx/emqx:v5.7.0 tagged

Copy link
Member

@zmstone zmstone left a comment

Choose a reason for hiding this comment

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

Missing information:

  1. Plugin config format (AVRO JSON)
  2. Plugin config file initial location
  3. Plugin run-time update REST API (maybe provide a curl example).
  4. UI aid is only available in Enterprise ediition.

@zmstone
Copy link
Member

zmstone commented May 9, 2024

Add some get_config example.
Also describe config life-cycle, when is config loaded, before plugin app start/2 callback or after etc.

rebar_template.config Outdated Show resolved Hide resolved
@zmstone
Copy link
Member

zmstone commented May 17, 2024

the default config should continue to be hocon.
Remember: hocon is super-set of JSON.

EMQX should type-check the config with avro schema if the schema exists. Otherwise just load it as-is

@JimMoen JimMoen force-pushed the feat/bump-avsc-example branch 3 times, most recently from f88b569 to 9c4aae2 Compare May 17, 2024 11:10
@JimMoen JimMoen force-pushed the feat/bump-avsc-example branch 3 times, most recently from 8730696 to ec3a307 Compare May 21, 2024 23:16
@JimMoen JimMoen changed the title feat: AVRO schema and i18n example feat: bump avsc and i18n example files May 22, 2024
@@ -3,20 +3,29 @@
{description, "Another amazing EMQX plugin."},
{version, "1.0.0", "The release version of this plugin."},
{app_vsn, "0.1.0", "The erlang application vsn value."},
{emqx_vsn, "v5.4.1", "EMQX version to use as a dependency."},
{emqx_vsn, "v5.6.1", "EMQX version to use as a dependency."},
Copy link
Member

Choose a reason for hiding this comment

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

nit:

Suggested change
{emqx_vsn, "v5.6.1", "EMQX version to use as a dependency."},
{emqx_vsn, "v5.6.1", "EMQX version to use as a dependency for compile and test."},

Copy link
Member Author

Choose a reason for hiding this comment

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

Will be updated when v5.7.0 released

@JimMoen JimMoen merged commit 37022d0 into master May 22, 2024
1 check passed
@zmstone zmstone deleted the feat/bump-avsc-example branch May 22, 2024 08:08
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.

4 participants