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.

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

Jump to bottom

Fix18.0 29424 member api #29441

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from
Open

Fix18.0 29424 member api #29441

wants to merge 12 commits into from

Conversation

GregoryBlemand
Copy link

FIX #29424 part 18.0

Fix member and thirdparty fetch and bad error messages

@eldy eldy changed the base branch from 18.0 to develop April 24, 2024 18:41
@eldy
Copy link
Member

eldy commented Apr 24, 2024

The PR contains both a fix and refactoring.
Fix must contains the MINIMAL changes to fix the bug. Because it constains refactoring i pushed the PR into develop branch. But there is a lot of error reported by the CTI (like identation broken).

@eldy eldy added PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do) PR to fix - Continuous Integration Test failed The PHP unit tests return something wrong with our code. Check the details to know what to fix. and removed PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do) labels Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
PR to fix - Continuous Integration Test failed The PHP unit tests return something wrong with our code. Check the details to know what to fix.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

BUG: in API GET member categories
2 participants
@GregoryBlemand @eldy