Skip to content

[V4] Add Warnings about executeWith... Limitations#981

Merged
Johannes-Schneider merged 1 commit intoinnovation/v4from
v4/execute-with-limitations
Aug 28, 2022
Merged

[V4] Add Warnings about executeWith... Limitations#981
Johannes-Schneider merged 1 commit intoinnovation/v4from
v4/execute-with-limitations

Conversation

@Johannes-Schneider
Copy link
Contributor

What Has Changed?

This PR adds "danger boxes" about the current limitations with regard to the executeWith... API, especially in combination with the CAP integration.

Manual Checks?

  • Text adheres to the style guide (vale docs/)
    • Every sentence is on its own line
    • Headings use title capitalization (applies also to the sidebar and title of a document)
    • You followed naming center guidelines when referring to SAP products (e.g. SAP S/4HANA)
  • You checked your spelling and grammar (consider using Grammarly, see CONTRIBUTING.md)
  • You formatted all changed files with prettier (npm run prettier)
  • You tested if the documentation still builds (npm run build)
  • You verified all new and changed links still work (changing the id or name of a file can break links)
  • You have updated the feature matrix if you add documentation on a new feature or otherwise required.

@Johannes-Schneider Johannes-Schneider added the Java Related to documentation on the SAP Cloud SDK for Java label Aug 26, 2022
@MatKuhr
Copy link
Member

MatKuhr commented Aug 26, 2022

we could mention that it might depend on the specific facades, e.g. execute with passport should still work, but that adds to the confusion and I'd rather go with the more clear, general statement

@Johannes-Schneider Johannes-Schneider merged commit deb98da into innovation/v4 Aug 28, 2022
@Johannes-Schneider Johannes-Schneider deleted the v4/execute-with-limitations branch August 28, 2022 11:53
Johannes-Schneider added a commit that referenced this pull request Oct 14, 2022
* [ThreadContext] Update Async Execution Docs (#852)

* Update Async Execution Docs

* Formatting

* Update docs/java/features/multi-tenancy/thread-context.mdx

Co-authored-by: Johannes Schneider <[email protected]>

* Update docs/java/features/multi-tenancy/thread-context.mdx

Co-authored-by: Johannes Schneider <[email protected]>

* Update docs/java/features/multi-tenancy/thread-context.mdx

Co-authored-by: Johannes Schneider <[email protected]>

* Update docs/java/features/multi-tenancy/thread-context.mdx

Co-authored-by: Johannes Schneider <[email protected]>

* Update docs/java/features/multi-tenancy/thread-context.mdx

Co-authored-by: Johannes Schneider <[email protected]>

* Add alternative

* Refine Executor Customization

* Formatting

* Update docs/java/features/multi-tenancy/thread-context.mdx

Co-authored-by: Johannes Schneider <[email protected]>

* Extend example

* Formatting

* Formatting

* Slight rewording

* Update docs/java/features/multi-tenancy/thread-context.mdx

Co-authored-by: Charles Dubois <[email protected]>

Co-authored-by: Johannes Schneider <[email protected]>
Co-authored-by: Charles Dubois <[email protected]>

* remove mentions of the RequestScopedHttpClientCache

* Revert "remove mentions of the RequestScopedHttpClientCache"

This reverts commit ec3481d.

* remove mentions of the RequestScopedHttpClientCache (#865)

* remove mentions of the service sdk (#892)

* Remove deprecated strategy (#956)

* Java/actually v4/mention sjb bom (#968)

* Mention SJB BOMs in dependency management chapter

* Add section on how to locally run archetypes

* Fix vale findings

* Add some links to the SJB BOM documentation

* Moar Links\!

* Add archetype clarification

* Review

* Update docs-java/guides/dependencies.mdx

Co-authored-by: Matthias Kuhr <[email protected]>

Co-authored-by: Matthias Kuhr <[email protected]>

* add danger boxes for CAP integration (#981)

* [V4] Thread Context Decorator Update (#993)

* Update Async Docs

* Add note on CDS version

* Update docs-java/features/multi-tenancy/thread-context.mdx

Co-authored-by: Johannes Schneider <[email protected]>

* Remove destination mocking from docs

* replace removed strategy (#1032)

* Minor updates for 4.0 and SJB Docs Tweaks

* adjust usage instructions for the RequestHeaderContainer (#1041)

* Added version dropdown (#1048)

* Fixed Vale warnings

* Fixed errors

* [Java] 4.0 Update Guide (#1046)

* First Draft

* Formatting

* Streamline versions

* Update docs-java/guides/4.0-upgrade.mdx

Co-authored-by: KavithaSiva <[email protected]>

* fix

* Extend deprecations table

* Finalise Guide + Update Release Notes

* Add stabilisation notes

* Integrate with versioned docs

Co-authored-by: KavithaSiva <[email protected]>
Co-authored-by: Johannes Schneider <[email protected]>
Co-authored-by: Charles Dubois <[email protected]>

* Vale off for 4.0-upgrade.mdx

* Removed placeholder release notes, fixed v3 sidebar

* Updated thread-context.mdx

* [Java V4] Release Notes (#1057)

* build(deps-dev): bump @typescript-eslint/eslint-plugin (#1054)

Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 5.39.0 to 5.40.0.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v5.40.0/packages/eslint-plugin)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps-dev): bump @typescript-eslint/parser from 5.39.0 to 5.40.0 (#1055)

Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 5.39.0 to 5.40.0.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v5.40.0/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps-dev): bump eslint-plugin-react from 7.31.9 to 7.31.10 (#1056)

Bumps [eslint-plugin-react](https://github.com/jsx-eslint/eslint-plugin-react) from 7.31.9 to 7.31.10.
- [Release notes](https://github.com/jsx-eslint/eslint-plugin-react/releases)
- [Changelog](https://github.com/jsx-eslint/eslint-plugin-react/blob/master/CHANGELOG.md)
- [Commits](jsx-eslint/eslint-plugin-react@v7.31.9...v7.31.10)

---
updated-dependencies:
- dependency-name: eslint-plugin-react
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* add release notes

* revert unrelated changes

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* add note about cve-2022-42003 (#1059)

* re-enable prettier for main java version (#1062)

* Formatting

* Formatting

* Update Release Policy document

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Matthias Kuhr <[email protected]>
Co-authored-by: Johannes Schneider <[email protected]>
Co-authored-by: Christoph Schubert <[email protected]>
Co-authored-by: Matthias Kuhr <[email protected]>
Co-authored-by: KavithaSiva <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Java Related to documentation on the SAP Cloud SDK for Java

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants