Skip to content

Removed empty array option for CredentialProducts in AppCredential file due to strict Type#398

Merged
kedarkhaire merged 2 commits intoapigee:4.xfrom
kedarkhaire:issue-397
Jan 21, 2025
Merged

Removed empty array option for CredentialProducts in AppCredential file due to strict Type#398
kedarkhaire merged 2 commits intoapigee:4.xfrom
kedarkhaire:issue-397

Conversation

@kedarkhaire
Copy link
Copy Markdown
Collaborator

@kedarkhaire kedarkhaire commented Dec 3, 2024

Fixes #397

In symfony 7, strict return type is expected as new TypeInfo is introduced.

returing stdClass instead of CredentialProduct class.

@kedarkhaire kedarkhaire changed the base branch from 3.x to 4.x December 3, 2024 13:55
@kedarkhaire kedarkhaire marked this pull request as ready for review January 17, 2025 08:14
@kedarkhaire kedarkhaire changed the title Changes for stdClass passed instead of CredentialProductInterface class Changes for CredentialProduct array Jan 17, 2025
@kedarkhaire kedarkhaire added this to the 4.0.0 milestone Jan 17, 2025
@kedarkhaire kedarkhaire changed the title Changes for CredentialProduct array Removed empty array for CredentialProduct in AppCredential file due to strict Type Jan 17, 2025
@kedarkhaire kedarkhaire changed the title Removed empty array for CredentialProduct in AppCredential file due to strict Type Removed empty array option for CredentialProducts in AppCredential file due to strict Type Jan 17, 2025
Copy link
Copy Markdown
Collaborator

@divya-intelli divya-intelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kedarkhaire kedarkhaire merged commit 28b1267 into apigee:4.x Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI issue for TypeError for Symfony 7

2 participants