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

Collection name should be the fully-qualified name of a Type, rather then just the simple name #20

Open
jakhog opened this issue Jan 17, 2019 · 2 comments
Assignees

Comments

@jakhog
Copy link
Contributor

jakhog commented Jan 17, 2019

If there is two ReadModels with the same name in different namespaces, the ReadModelRepositoryFor will be mapped to the same collection.

In the best case, that will currently result in an exception. But it might not...

┆Issue is synchronized with this Asana task

@smithmx smithmx self-assigned this Jan 17, 2019
@woksin woksin closed this as completed Jan 17, 2019
@woksin woksin reopened this Jan 17, 2019
@woksin
Copy link
Contributor

woksin commented Jan 17, 2019

Did not think about that, this screws up with something that we have at the CBS repo... Should we fix and patch this today, this should just be a 2min fix 😉
CC @jakhog

@endre-synnes
Copy link

What happened here @smithmx, we just had the same problem once again (from Jakob).

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

No branches or pull requests

4 participants