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

Added info about targetNamespace change in README #2559

Merged
merged 2 commits into from
Sep 7, 2022

Conversation

fballiano
Copy link
Contributor

Since #2539 it's probably necessary to have some info in our README file, this is it.

@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

addison74
addison74 previously approved these changes Sep 6, 2022
@sreichel
Copy link
Contributor

sreichel commented Sep 6, 2022

I'd add a new headline "important change to soap/wsdl" with some info how to fix it.

Sdfendor
Sdfendor previously approved these changes Sep 6, 2022
@fballiano fballiano force-pushed the fballiano-targetnamespace branch from ff86b68 to 966d98a Compare September 6, 2022 22:53
@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

@fballiano fballiano dismissed stale reviews from Sdfendor and addison74 via efaf3ec September 6, 2022 23:13
@fballiano
Copy link
Contributor Author

I've created a section with the text that I already added to the release notes

@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

@sreichel sreichel merged commit 49112dc into 1.9.4.x Sep 7, 2022
@sreichel sreichel deleted the fballiano-targetnamespace branch September 7, 2022 01:20
@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2022

Unit Test Results

1 files  ±0  1 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
7 runs  ±0  5 ✔️ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit 49112dc. ± Comparison against base commit a731ed8.

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.

5 participants