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

Discovery for Studio event bus use case #25

Closed
7 tasks done
robrap opened this issue Jun 13, 2022 · 6 comments
Closed
7 tasks done

Discovery for Studio event bus use case #25

robrap opened this issue Jun 13, 2022 · 6 comments
Assignees
Labels
event-bus Work related to the Event Bus.

Comments

@robrap
Copy link
Contributor

robrap commented Jun 13, 2022

This discovery is to both confirm that the Studio course publish event to Discovery use case, as a replacement for a part of Refresh Course Metadata, is the right move. It is also to help create follow-up tasks.

  • Document findings.
  • Confirm difficulty of publishing the course publish event in Studio. This would entail appropriate event design.
  • Confirm difficulty of consuming the event in Discovery and logging.
  • Confirm difficulty of consuming the event in Discovery and running appropriate idempotent code to update the course. (Determine if this should be left to another Epic?)
  • Who owns the Studio event publish side? TNL? We should discuss long-term ownership with them.
  • Does Pheonix own the Discovery consuming side? We should discuss long-term ownership with them.
  • Support creation of follow-up tickets (with discussion first), including moving boiler-plate code to proper libraries.
@robrap
Copy link
Contributor Author

robrap commented Jun 13, 2022

@rgraber: Can you also document any findings so far from any discovery you've done?

@robrap robrap added event-bus Work related to the Event Bus. and removed event-bus Work related to the Event Bus. labels Jun 13, 2022
@robrap robrap added the event-bus Work related to the Event Bus. label Jun 14, 2022
@rgraber rgraber moved this from In Progress to Done in Arch-BOM Jul 12, 2022
@timmc-edx timmc-edx moved this from Done to In Progress in Arch-BOM Sep 22, 2022
@rgraber
Copy link
Contributor

rgraber commented Sep 27, 2022

@robrap
Copy link
Contributor Author

robrap commented Sep 27, 2022

@rgraber: I think we discussed long-term ownership with Phoenix. I'm not sure what we did for TNL. They should be involved with sad-path of the event publishing, and what they think they'd do regarding sad-path, etc. Should we create a separate ticket, or just move this unchecked task to the sad-path task so we can close this out?

@rgraber
Copy link
Contributor

rgraber commented Sep 29, 2022

I think either way "Who owns the Studio event publish side? TNL? We should discuss long-term ownership with them" does not work as an acceptance criteria. I'm not even sure it belongs in a ticket.

@rgraber
Copy link
Contributor

rgraber commented Sep 29, 2022

I think we should close this. We can put a note in the sad path task to make sure to run it by TNL.

@robrap
Copy link
Contributor Author

robrap commented Sep 29, 2022

Thanks. I added a note about TNL to #54 (sad path). I want this represented in a ticket because I think we'll get into trouble if we don't address it, and it would be easy to forget about.

@robrap robrap closed this as completed Sep 29, 2022
Repository owner moved this from In Progress to Done in Arch-BOM Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
event-bus Work related to the Event Bus.
Projects
Archived in project
Development

No branches or pull requests

2 participants