Skip to content

Prepare links for migration - #23

Merged
JeremyMcCormick merged 4 commits into
mainfrom
iss22
May 22, 2024
Merged

JeremyMcCormick merged 4 commits into
mainfrom
iss22

Conversation

@gmantele

Copy link
Copy Markdown
Member

This PR updates links of the sitemap (list of links available at the bottom of all website pages). All links (except Wiki and Documents links) are now made relative to this prototype website.

Don't hesitate to tell me if I forgot some or if I fixed the wrong one or in the wrong way.

FIxes #22

@gmantele gmantele self-assigned this May 21, 2024
@gmantele gmantele mentioned this pull request May 21, 2024
@gmantele

Copy link
Copy Markdown
Member Author

I may still change one breaking thing:

  • move the content of the content/pages/ directory to the upper level (i.e. content/)
  • remove the pages/ directory

Otherwise, URLs would be https://ivoa.net/pages/about/....which would be weird and would break any bookmark on existing pages that people might have kept on their machine. So, with this last modification, we would have the URL: https://ivoa.net/about/.

Just tell me if it is a bad idea.

@gmantele

Copy link
Copy Markdown
Member Author

Ok.According to me, this PR is now ready for review and to be merged.

Comment on lines +29 to +34
<li><a href="{{ .Site.Home.Permalink}}/astronomers/getting_started/">Getting Started</a></li>
<li><a href="{{ .Site.Home.Permalink}}/astronomers/using_the_vo/">Using the VO</a></li>
<li><a href="{{ .Site.Home.Permalink}}/astronomers/vo_glossary/">VO Glossary</a></li>
<li><a href="{{ .Site.Home.Permalink}}/astronomers/applications/">VO Applications</a></li>
<li><a href="{{ .Site.Home.Permalink}}/newsletter/">IVOA newsletter</a></li>
<li><a href="{{ .Site.Home.Permalink}}/astronomers/vo_for_public/">VO for Students & Public</a></li>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think these all have an extra "/" between the {{ .Site.Home.Permlink}} and the folder. It works in the browser anyway, but maybe we can remove it.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, I have noticed that too. As it does not seem to be an issue, I have not put an high priority to solve this yet. But I can have a quick look now, if you want to.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

no, definitely no need to prioritise this, let's just keep it in mind

@molinaro-m molinaro-m left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Besides the small comment, I thin this is fine.
I checked the version's outcome on the webstage and looks fine.

@JeremyMcCormick JeremyMcCormick left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me. The link for the one ported page works fine and the others are 404 now but I assume that is intended here so we can work on adding the new pages that are pointed to now.

We need to update a few things in the contributing guide based on these changes, but I can do that on another branch.

@JeremyMcCormick
JeremyMcCormick merged commit 432cd5f into main May 22, 2024
@JeremyMcCormick
JeremyMcCormick deleted the iss22 branch April 30, 2025 01:46
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.

Prepare links in the bottom part

3 participants