Skip to content

docs: fix broken link in contributor guide - #1785

Merged
EnriqueL8 merged 2 commits into
hyperledger-firefly:mainfrom
Popie52:fix-docs-link
Sep 24, 2026
Merged

EnriqueL8 merged 2 commits into
hyperledger-firefly:mainfrom
Popie52:fix-docs-link

Conversation

@Popie52

@Popie52 Popie52 commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

Proposed changes

This PR includes a few minor documentation and typo fixes:

  1. Fixes an outdated URL in contributors/index.md that was pointing to a non-existent /docs directory instead of the correct doc-site/docs directory.
  2. Fixes capitalization for macOS in releasenotes/index.md.
  3. Fixes a grammatical error in subscription_description.md.
  4. Adds the missing tutorials/chains/index.md file.

Fixes #


Types of changes

  • Documentation Update

Please make sure to follow these points

  • I have read the contributing guidelines.
  • I have performed a self-review of my own code or work.
  • I have commented my code, particularly in hard-to-understand areas.
  • My changes generates no new warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • My changes have sufficient code coverage (unit, integration, e2e tests).

Screenshots (If Applicable)

Screenshot From 2026-09-19 15-37-48

Other Information

These are minor, low-risk documentation fixes.

@Popie52
Popie52 requested a review from a team as a code owner September 19, 2026 10:10
@Popie52

Popie52 commented Sep 19, 2026

Copy link
Copy Markdown
Contributor Author

@EnriqueL8 can u have a look at it

@EnriqueL8 EnriqueL8 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @Popie52 a few comments

Comment thread doc-site/docs/contributors/index.md
@@ -1,4 +1,4 @@
Each [Subscription](#subscription) tracks delivery of events to a particular
Each Subscription tracks delivery of events to a particular

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why did you change this?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Popie52 just this one missing?

Comment thread doc-site/docs/reference/namespaces.md Outdated
will interact with the first contract in the list until instructions are received to terminate it and
migrate to the next.

## Adding a Namespace with the FireFly Helm Chart

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

you seem to have bundled the namespace change here

Comment thread doc-site/docs/tutorials/chains/index.md Outdated
FireFly can connect to a variety of blockchain networks through its pluggable
blockchain connector framework.

The tutorials below provide instructions for connecting a FireFly development

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Remove development environment

Signed-off-by: popie52 <popiesailor@gmail.com>
@codecov

codecov Bot commented Sep 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@EnriqueL8 EnriqueL8 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @Popie52

@EnriqueL8
EnriqueL8 merged commit 18e4336 into hyperledger-firefly:main Sep 24, 2026
19 checks passed
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