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

Fix Wrong facet function called when defining a custom group #7654 #7994

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vishal-agr99
Copy link
Contributor

Fixes #
Wrong facet function called when defining a custom group is fixed in this PR.
#7654

@Labuschagne-Miro
Copy link
Contributor

@Zharktas I am working on Pull request #7754. It looks this pull request is copied from #7754

@vishal-agr99
Copy link
Contributor Author

@Zharktas I am working on Pull request #7754. It looks this pull request is copied from #7754

@Labuschagne-Miro
If you want to work on any issue, Please make a comment on that issue so that other developer will understand that someone is working.

BTW This PR is not copied, In the last PR Tests are failing
but here no tests failed.

Thanks for your understanding.

@Labuschagne-Miro
Copy link
Contributor

@Vishal-NEC I have commented on pull request #7754 before you have raise this pull request. You should first confirm with author and community member before working on any issue.

Secondly, All test cases are passing doesn't mean your pull request is complete enough to fix the issue.

@Labuschagne-Miro
Copy link
Contributor

@Zharktas , What is your opinion on this pull request?

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

Successfully merging this pull request may close these issues.

None yet

3 participants