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

Changes to support the redefining of a namespace alias within different elements. #1151

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

Conversation

lah8789
Copy link

@lah8789 lah8789 commented Jul 23, 2021

The provided WSDL was not serializing the response correctly when returning the XML.
Created a request response folder to test the changes that were made and made sure code coverage remained the same.

Commented out any test referencing json_response.wsdl because it was failing after I cloned master and ran the tests.

Copy link
Collaborator

@jsdevel jsdevel left a comment

Choose a reason for hiding this comment

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

@lah8789 all current tests on the master branch pass for me. we try to stay back compat on this project. please leave existing tests alone and add new ones.

@lah8789 lah8789 force-pushed the namespace-redefined-in-element branch from 6211ef5 to 22c8c88 Compare July 27, 2021 15:07
within different elements.  The provided WSDL was not serializing the response
correctly when returning the XML.
Created a request response folder to test the changes that were made and made sure
code coverage remained the same.
@lah8789 lah8789 force-pushed the namespace-redefined-in-element branch from dedd6dc to 3c88c5e Compare July 27, 2021 15:25
@lah8789
Copy link
Author

lah8789 commented Jul 27, 2021

@jsdevel I have reverted back changes to tests and only have the original changes related to the PR

@lah8789 lah8789 force-pushed the namespace-redefined-in-element branch from 3c88c5e to cfb4497 Compare July 27, 2021 15:34
@lah8789 lah8789 requested a review from jsdevel July 27, 2021 15:37
@lah8789
Copy link
Author

lah8789 commented Sep 15, 2021

@jsdevel , have you had a chance to look at this?

@jsdevel
Copy link
Collaborator

jsdevel commented Oct 1, 2021

@lah8789 please pull the latest from master so the PR check can run

@lah8789
Copy link
Author

lah8789 commented Oct 7, 2021

@lah8789 please pull the latest from master so the PR check can run

@jsdevel, This has been completed

@jsdevel
Copy link
Collaborator

jsdevel commented Nov 3, 2021

@lah8789 please ensure all tests pass.

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.

2 participants