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

docs: fix wrong link for API credentials #913

Merged
merged 1 commit into from
Oct 1, 2024
Merged

docs: fix wrong link for API credentials #913

merged 1 commit into from
Oct 1, 2024

Conversation

lechnerc77
Copy link
Member

@lechnerc77 lechnerc77 commented Oct 1, 2024

Purpose

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[X] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Test the code via automated test
go test ./...

What to Check

Verify that the following are valid:

  • Automated tests are executed successfully
  • Check markdown to contain same link as help.sap.com

Other Information

Validated layout of markdown files at https://registry.terraform.io/tools/doc-preview

Checklist for reviewer

The following organizational tasks must be completed before merging this PR:

  • The PR is assigned to the Terraform project and a status is set (typically "in review").
  • The PR has the matching labels assigned to it.
  • The PR has a milestone assigned to it.
  • If the PR closes an issue, the issue is referenced.
  • Possible follow-up items are created and linked.

@lechnerc77 lechnerc77 self-assigned this Oct 1, 2024
@lechnerc77 lechnerc77 added the documentation Improvements or additions to documentation label Oct 1, 2024
@lechnerc77 lechnerc77 added this to the 1.8.0 milestone Oct 1, 2024
@lechnerc77 lechnerc77 enabled auto-merge (squash) October 1, 2024 05:35
Copy link
Contributor

@diya-dhan diya-dhan left a comment

Choose a reason for hiding this comment

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

Changes look fine to me

@lechnerc77 lechnerc77 merged commit 20d4aee into main Oct 1, 2024
16 checks passed
@lechnerc77 lechnerc77 deleted the fix/issue-912 branch October 1, 2024 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Incorrect Documentation Link for "Further Documentation" in btp_subaccount_api_credential Resource
2 participants