Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 PHP 8.2 and Other Related Issues #640
base: master
Are you sure you want to change the base?
Fix PHP 8.2 and Other Related Issues #640
Changes from 11 commits
76dc113
591629c
b81fb0b
fad9d56
f4e69f4
c6138da
d580082
959b1a7
475e87f
dd38b0f
6da6e4f
6c17547
a096ea6
ba9a468
96d67ae
c1f8164
ce7d988
87bfb35
c31570d
a12582a
aa87640
156a26d
089bff7
964ee7e
596cca5
cfe79c5
d77e72b
85d3080
27520ac
d3f2448
6dc14ea
0149fb7
f205667
275dfda
87f0929
ed80f9f
6b9f10f
f5303e0
3f3e135
91ca58a
586b770
0268830
beac573
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Large diffs are not rendered by default.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason for adding this to the PayIn class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This property was identified in both the documentation and PHP SDK. I verified its existence through debug logs and implemented it accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason for adding this to the PayIn class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This property was identified in both the documentation and PHP SDK. I verified its existence through debug logs and implemented it accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason for adding this to the PayIn class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This property was identified in both the documentation and PHP SDK. I verified its existence through debug logs and implemented it accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason for adding this to the PayIn class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This property was identified in both the documentation and PHP SDK. I verified its existence through debug logs and implemented it accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason for adding this to the PayIn class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This property was identified in both the documentation and PHP SDK. I verified its existence through debug logs and implemented it accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason for adding this to the PayIn class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This property was identified in both the documentation and PHP SDK. I verified its existence through debug logs and implemented it accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you add this param? I don't see it in the docs (correct me if I'm wrong)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This property was identified in both the documentation and PHP SDK. I verified its existence through debug logs and implemented it accordingly.