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.

Sign up for GitHub

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

Support integer, atom or strings as aggregate_uuid in Commanded.Aggregates.Registry.open_aggregate(aggregate_module, aggregate_uuid) #7

Merged

slashdotdash merged 2 commits into commanded:master from andrzejsliwa:master

Sep 22, 2016

Contributor

andrzejsliwa commented Sep 21, 2016 •

edited

Loading

lets integer, atom or strings as aggregate_uuid in

Commanded.Aggregates.Registry.open_aggregate(aggregate_module, aggregate_uuid)

using other values than string raising low level error on postgrex driver about text type

andrzejsliwa added 2 commits

September 21, 2016 23:24


let use integer, atom or strings as aggregate_uuid

68a1eb5


cleanup of unused alias and variable

0b86e8c

andrzejsliwa mentioned this pull request

Aggregate UUID must be a string #6

Closed

slashdotdash changed the title ~~integer, atom or strings as aggregate_uuid in Commanded.Aggregates.Registry.open_aggregate(aggregate_module, aggregate_uuid)~~ Support integer, atom or strings as aggregate_uuid in Commanded.Aggregates.Registry.open_aggregate(aggregate_module, aggregate_uuid)

slashdotdash merged commit 7f6b9e6 into commanded:master

Member

slashdotdash commented Sep 22, 2016

Great, thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment