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

cds.test with authentication #631

Merged
merged 10 commits into from
Jan 26, 2024
Merged

cds.test with authentication #631

merged 10 commits into from
Jan 26, 2024

Conversation

larslutz96
Copy link
Contributor

Our documentation nicely explains how to use service APIs and HTTP APIs with cds.test.

We should add a link to the authentication configuration and explain which kind is used for cds.test. In addition, we should add samples how to set a user for the above mentioned APIs.

Our documentation nicely explains how to use service APIs and HTTP APIs with cds.test.

We should add a link to the authentication configuration and explain which kind is used for cds.test. In addition, we should add samples how to set a user for the above mentioned APIs.
Copy link
Member

@chgeo chgeo left a comment

Choose a reason for hiding this comment

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

Nice. See comments.

node.js/cds-test.md Outdated Show resolved Hide resolved
node.js/cds-test.md Outdated Show resolved Hide resolved
larslutz96 and others added 2 commits January 16, 2024 15:31
Co-authored-by: Christian Georgi <[email protected]>
Co-authored-by: Christian Georgi <[email protected]>
@renejeglinsky
Copy link
Contributor

Closes: #371

node.js/cds-test.md Outdated Show resolved Hide resolved
node.js/cds-test.md Outdated Show resolved Hide resolved
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Spelling Mistakes

  • ./node.js/cds-test.md:86:237 Unknown word "documenation"

Generally, for each spelling mistake there are 2 ways to fix it:

  1. Fix the spelling mistake and commit it.
  2. The word is incorrectly reported as misspelled → put the word on the project-words.txt list, located in the root project directory.

node.js/cds-test.md Outdated Show resolved Hide resolved
larslutz96 and others added 2 commits January 24, 2024 14:38
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
node.js/cds-test.md Outdated Show resolved Hide resolved
node.js/cds-test.md Outdated Show resolved Hide resolved
Copy link
Contributor

@renejeglinsky renejeglinsky left a comment

Choose a reason for hiding this comment

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

Please check if my last edit is fine for both of you.

@larslutz96
Copy link
Contributor Author

Please check if my last edit is fine for both of you.

@johannes-vogel seems good to me

@renejeglinsky renejeglinsky merged commit 0fef917 into main Jan 26, 2024
4 checks passed
@renejeglinsky renejeglinsky deleted the larslutz96-patch-1 branch January 26, 2024 09:17
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.

4 participants