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 update] Correct "How to assign a dataset identifier" link #277

Open
pzwsk opened this issue Sep 10, 2024 · 3 comments
Open

[Docs update] Correct "How to assign a dataset identifier" link #277

pzwsk opened this issue Sep 10, 2024 · 3 comments
Assignees
Labels
Docs This issue relates to documentation

Comments

@pzwsk
Copy link
Contributor

pzwsk commented Sep 10, 2024

If you are proposing changes or updates to the RDL documentation, then please fill in the below template.

What is the context or reason for the change?

When on this page https://docs.riskdatalibrary.org/en/latest/reference/browser/ and looking at id field, it is written : For more information, see how to assign a dataset identifier.

However, when clicking on the link it brings you to the Codelists page. The correct link should be https://docs.riskdatalibrary.org/en/latest/guides/metadata/#assign-a-dataset-identifier

What is your proposed change?

In https://docs.riskdatalibrary.org/en/latest/reference/browser/
replace "how to assign a dataset identifier" url https://rdl-standard.readthedocs.io/en/latest/reference/codelists/#assign-a-dataset-identifier
with https://docs.riskdatalibrary.org/en/latest/guides/metadata/#assign-a-dataset-identifier

@pzwsk pzwsk added the Docs This issue relates to documentation label Sep 10, 2024
@pzwsk pzwsk self-assigned this Sep 10, 2024
@pzwsk
Copy link
Contributor Author

pzwsk commented Sep 10, 2024

Correction:

change should be made in the rdls schema itself (line 22)

https://github.com/GFDRR/rdl-standard/blob/main/schema/rdls_schema.json

@pzwsk
Copy link
Contributor Author

pzwsk commented Sep 10, 2024

@duncandewhurst or @matamadio appreciate if you could have a quick look to confirm

@matamadio
Copy link
Contributor

Agree on the proposed change.

pzwsk added a commit that referenced this issue Sep 13, 2024
@pzwsk pzwsk mentioned this issue Sep 13, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs This issue relates to documentation
Projects
None yet
Development

No branches or pull requests

3 participants