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

fix: update the link to Chart Parameters section #159

Merged
merged 2 commits into from
Oct 10, 2024

Conversation

vil02
Copy link
Contributor

@vil02 vil02 commented Oct 2, 2024

Updates the link in openfga/README.

Description

This line:

This will deploy a 3-replica deployment of OpenFGA on the Kubernetes cluster using the default configurations for OpenFGA. For more information on the default values, please see the official [OpenFGA documentation](https://openfga.dev/docs/getting-started/setup-openfga/docker#configuring-the-server). The [Parameters](#parameters) section below lists the parameters that can be configured during installation.

contains a broken link. This PR fixes it.

References

For simplicity I decided not to create an issue for that (it would be an overkill for such typo-like fix). Please let me know if this is fine.

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

@vil02 vil02 requested review from a team as code owners October 2, 2024 12:44
@rhamzeh rhamzeh enabled auto-merge (squash) October 10, 2024 16:23
@rhamzeh
Copy link
Member

rhamzeh commented Oct 10, 2024

Thanks @vil02

@rhamzeh rhamzeh merged commit 20c3505 into openfga:main Oct 10, 2024
2 of 4 checks passed
@vil02 vil02 deleted the fix_broken_link_to_chart-parameters branch October 10, 2024 18:38
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