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

Fixing links #130

Merged
merged 8 commits into from
Jun 5, 2024
Merged

Fixing links #130

merged 8 commits into from
Jun 5, 2024

Conversation

CsatariGergely
Copy link
Contributor

No description provided.

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Configuration-as-Data definition was copied from
https://github.com/kptdev/kpt/blob/main/docs/design-docs/06-config-as-data.md#configuration-as-data-1

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
@nephio-prow nephio-prow bot requested review from henderiw and liamfallon May 30, 2024 12:23
Copy link

netlify bot commented May 30, 2024

Deploy Preview for nephio ready!

Name Link
🔨 Latest commit d698826
🔍 Latest deploy log https://app.netlify.com/sites/nephio/deploys/665d74a154412f0008b5bb94
😎 Deploy Preview https://deploy-preview-130--nephio.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

linkspector can not follow the links of images as they are
stored in the static directory, but still served from the root path.

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Exclude pattersn for .png images was mailformed

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
- Using a replace pattersn instead of ignore for png images
- Fixing more links

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
manipulating configuration data don’t need to directly interact with storage (git, container images)

Source of definition and more information about Configuration as Data can be found in the
[kpt documentation](https://github.com/kptdev/kpt/blob/main/docs/design-docs/06-config-as-data.md#configuration-as-data-1).
Copy link
Contributor

Choose a reason for hiding this comment

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

I know the porch docs are not "ready" but should we link to them instead?
https://github.com/nephio-project/porch/blob/main/docs/06-config-as-data.md

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, yes, definitely. For me it is not really clear where KPT ends and where Porch starts in the documentation.
On the mid term I would like to move the Porch documentation under the Nephio docs (nephio-project/nephio#744). On long term the Porch documentation should follow the code of Porch+

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in d698826

@@ -369,6 +369,12 @@ The new cluster comes up after a number of minutes.

# Transformations in the Workload cluster creation

Test mest
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a filler/placeholder?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, yes, thanks for catching this. I've realized that Docsy does not render the H1 headings to the right side page content list. This is remained after the debugging of the issue. I will try to figure out how to deal with H1 headings and create a PR to fix them in the whole docs set.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in d698826

- Pointing to the "new" Porch documentation in the Nephio org
- Removal of test text

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
@arora-sagar
Copy link
Contributor

@efiacor Can we merge this MR? I suppose it will fix the errors in other MRs.

@CsatariGergely
Copy link
Contributor Author

/assign @efiacor @liamfallon

Copy link
Member

@liamfallon liamfallon left a comment

Choose a reason for hiding this comment

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

Thanks for fixing the links.

@liamfallon
Copy link
Member

/lgtm
/approve

@nephio-prow nephio-prow bot added the lgtm label Jun 5, 2024
Copy link
Contributor

nephio-prow bot commented Jun 5, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: liamfallon

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@nephio-prow nephio-prow bot added the approved label Jun 5, 2024
@nephio-prow nephio-prow bot merged commit a6d278c into nephio-project:main Jun 5, 2024
8 checks passed
@CsatariGergely CsatariGergely deleted the fixing-links branch June 6, 2024 04:48
This was referenced Jun 12, 2024
nephio-prow bot pushed a commit that referenced this pull request Jun 13, 2024
Fixing the 404 issue of the breadcrumb navigation in R2.

/assign @efiacor @liamfallon 

This supposed to fix the issue for the R2 branch pointed out by @efiacor
in
#128 (review)

As an extra I'm adding the link fixes to R2 merged to main in
#130
(69c79e2)

Link checks for the [LFN Wiki](https://wiki.lfnetworking.org/) are
disabled as the wiki is down. [LF IT
ticket](https://jira.linuxfoundation.org/plugins/servlet/desk/portal/2/IT-26874)
was created.

---------

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
nephio-prow bot pushed a commit that referenced this pull request Jun 13, 2024
Fixing the 404 issue of the breadcrumb navigation in R3.

/assign @efiacor @liamfallon 

This supposed to fix the issue for the R3 branch pointed out by @efiacor
in
#128 (review)

As an extra I'm adding the link fixes to R3 merged to main in
#130
(69c79e2)

Link checks for the [LFN Wiki](https://wiki.lfnetworking.org/) are
disabled as the wiki is down. [LF IT
ticket](https://jira.linuxfoundation.org/plugins/servlet/desk/portal/2/IT-26874)
was created.

---------

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants