Skip to content
This repository has been archived by the owner on Jan 27, 2021. It is now read-only.

Metadata context #56

Merged
merged 3 commits into from
Aug 25, 2020
Merged

Metadata context #56

merged 3 commits into from
Aug 25, 2020

Conversation

kulmann
Copy link
Member

@kulmann kulmann commented Aug 25, 2020

Use go-micro's metadata context for passing around the account id in the context.

When this gets merged, we need a new 2.4.0 release.

@kulmann kulmann self-assigned this Aug 25, 2020
@kulmann kulmann marked this pull request as ready for review August 25, 2020 11:52
@kulmann kulmann requested review from butonic and IljaN August 25, 2020 11:52
Copy link
Contributor

@C0rby C0rby left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Marked UUIDKey as deprecated. Use AccountID instead if possible.
Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍 as discussed to keep the old one for now

@kulmann kulmann merged commit f0aa9e4 into master Aug 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants