bamaer commented on PR #8978:
URL: https://github.com/apache/hop/pull/8978#issuecomment-6230860143
Thanks @Adalennis, good catch on the missing navigation entry, and keeping
the list in one place makes sense.
Docs only, so nothing here can break anything. One thing worth fixing before
merge:
**Worth fixing**
- Four pages still link here as a full list of transforms, and that list is
gone after this PR:
- `getting-started/hop-next-steps.adoc:32` "the complete list of
transforms"
- `pipeline/create-pipeline.adoc:82` "Check the list of transforms"
- `pipeline/pipelines.adoc:176` "Transforms Catalog — Browse the complete
library of over 100 transforms."
- `snippets/hop-concepts/transform.adoc:21` "See the Transforms catalog
for all available transforms."
Suggested fix: reword these so they no longer promise a list. For example,
point them to "the transform pages under Pipelines › Transforms", or just link
to "Transforms".
Checked against the source tree, site not built: the 28 missing transforms,
that Beam Hive catalog input was the only transform page missing from the
navigation, that nothing includes the removed list, and that the PR merges
cleanly with main.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
