Skip to content
This repository has been archived by the owner on Oct 23, 2023. It is now read-only.

pass filter option payload - fix #230

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

pass filter option payload - fix #230

wants to merge 4 commits into from

Conversation

mustafahsyed
Copy link

@mustafahsyed mustafahsyed commented Nov 18, 2022 •
edited
Loading

Description

Related issues

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update (this needs a follow up PR)

Changes Made

Testing

  • Unit Tests
  • Integration Tests
  • Tests do not apply
  • Needs testing (start an issue or do a follow up PR about it)

Mentions

@mustafahsyed
Copy link
Author

pass filter option payload - fix

blankdots
blankdots suggested changes Dec 5, 2022
View reviewed changes
tests/aggregator/test_aggregator_utils.py Outdated Show resolved Hide resolved
registry/config/config.ini Outdated Show resolved Hide resolved
registry/config/config.ini Outdated Show resolved Hide resolved
registry/config/config.ini Outdated Show resolved Hide resolved
docs/conf.py Outdated Show resolved Hide resolved
aggregator/config/__init__.py Outdated Show resolved Hide resolved
aggregator/config/__init__.py Outdated Show resolved Hide resolved
aggregator/aggregator.py Outdated Show resolved Hide resolved
aggregator/aggregator.py Outdated Show resolved Hide resolved
aggregator/aggregator.py Outdated Show resolved Hide resolved
@mustafahsyed
Copy link
Author

Hi,

I have made all changes requested above, please have a look at my latest commit.
312b77f

Cheers
Mustafa

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

@blankdots blankdots blankdots requested changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mustafahsyed @blankdots