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

Prepare links for migration #23

Merged
merged 4 commits into from
May 22, 2024
Merged

Prepare links for migration #23

merged 4 commits into from
May 22, 2024

Conversation

gmantele
Copy link
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 added the new feature New feature or request label May 21, 2024
@gmantele gmantele self-assigned this May 21, 2024
@gmantele gmantele mentioned this pull request May 21, 2024
@gmantele
Copy link
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
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
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
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
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

Copy link
Member

@molinaro-m molinaro-m left a comment

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.

Copy link
Collaborator

@JeremyMcCormick JeremyMcCormick left a comment

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
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Prepare links in the bottom part
3 participants