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

Refactor read models subscriptions filter #600

Closed
adriangmweb opened this issue Feb 4, 2021 · 0 comments · Fixed by #662
Closed

Refactor read models subscriptions filter #600

adriangmweb opened this issue Feb 4, 2021 · 0 comments · Fixed by #662
Assignees
Labels
feature-request New feature or request package:core Affects the core package size: M Tasks where you have to change some files.

Comments

@adriangmweb
Copy link
Contributor

Feature Request

Refactor read-model-pub-sub.ts file to be able to use filters included on the GraphQL subscription queries

Description

Right now the new filters can't be tested properly to work on this file until the new GraphQL schema is created.

Possible Solution

Use the new filter method to test and refactor the file in order to work properly, also including tests.

Additional information

This issue will be solved once the new GraphQL read models filter is finished.

@adriangmweb adriangmweb added feature-request New feature or request size: M Tasks where you have to change some files. package:core Affects the core package labels Feb 4, 2021
@adriangmweb adriangmweb self-assigned this Feb 4, 2021
@adriangmweb adriangmweb mentioned this issue Feb 4, 2021
3 tasks
@adriangmweb adriangmweb mentioned this issue Mar 8, 2021
3 tasks
@adriangmweb adriangmweb linked a pull request Mar 8, 2021 that will close this issue
3 tasks
@adriangmweb adriangmweb mentioned this issue Apr 13, 2021
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request New feature or request package:core Affects the core package size: M Tasks where you have to change some files.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant