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

fix: add required parameter to produce_event command #133

Merged
merged 6 commits into from
Feb 16, 2023

Conversation

rgraber
Copy link
Contributor

@rgraber rgraber commented Feb 16, 2023

When we upgraded openedx_events to the version where event_metadata was required, we forgot to update the produce_event command, which is only used for testing. This just creates and EventsMetadata object from the event type and passes it.

Merge checklist:
Check off if complete or not applicable:

  • Version bumped
  • Changelog record added
  • Documentation updated (not only docstrings)
  • Commits are squashed
  • Noted any: Concerns, dependencies, deadlines, tickets, testing instructions

CHANGELOG.rst Outdated Show resolved Hide resolved
@rgraber rgraber merged commit 8836d12 into main Feb 16, 2023
@rgraber rgraber deleted the rsgraber/20230216-fix-produce-event-cmd branch February 16, 2023 20:56
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.

3 participants