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

Fix base.html URLs #379

Merged
merged 3 commits into from
Mar 23, 2023
Merged

Fix base.html URLs #379

merged 3 commits into from
Mar 23, 2023

Conversation

Mahhheshh
Copy link
Contributor

Solves #371
Updated the URLs used in the footer of the application to use the correct links. Previously, the links were pointing to the wrong resources, but have been updated to point to the correct URLs. This includes updating the links to the AiiDAlab registry page in the aiidalab-registry repository. These changes ensure that users can access the correct resources from the footer of the application.

This pull request fixes an issue in the parent file base.html at

{% set app_registry_url = "http://aiidateam.github.io/aiida-registry" %}
which was causing issues in the child file layout.html at https://github.com/aiidalab/aiidalab-registry/blob/8bc5d00fffa2016f562d065d81bd5a111c99479e/src/templates/layout.html#L8. The issue was causing variables to not be overwritten in the registry repository. The changes made in this pull request fix the links in the footer, and ensure that the variables can be correctly overwritten in the child file.

Updated the URLs used in the footer of the application to use the correct links. Previously, the links were pointing to the wrong resources, but have been updated to point to the correct URLs. This includes updating the links to the AiiDAlab registry page in the aiidalab-registry repository. These changes ensure that users can access the correct resources from the footer of the application.
Copy link
Member

@yakutovicha yakutovicha left a comment

Choose a reason for hiding this comment

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

thanks, @Mahhheshh. Could you also fix the links in: docs/source/conf.py and docs/source/usage/index.rst? They still point to the old AiiDAlab URL.

@Mahhheshh
Copy link
Contributor Author

Yes, are there any additional links I'm missing?

@yakutovicha
Copy link
Member

Yes, are there any additional links I'm missing?

I think that is all.

@codecov
Copy link

codecov bot commented Mar 23, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (50498b0) 59.98% compared to head (d9ab46b) 59.98%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #379   +/-   ##
=======================================
  Coverage   59.98%   59.98%           
=======================================
  Files          12       12           
  Lines        1122     1122           
=======================================
  Hits          673      673           
  Misses        449      449           
Flag Coverage Δ
py-3.10 59.98% <ø> (ø)
py-3.11 59.98% <ø> (ø)
py-3.8 59.98% <ø> (ø)
py-3.9 59.98% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@yakutovicha yakutovicha left a comment

Choose a reason for hiding this comment

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

Thanks, just a few small changes.

@@ -9,7 +9,7 @@ Access AiiDAlab

As a user, you have three options to access AiiDAlab:

1. Log into one of the `open AiiDAlab servers <https://materialscloud.org/aiidalab>`_.
1. Log into one of the `open AiiDAlab servers <https://www.aiidalab.net/>`_.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
1. Log into one of the `open AiiDAlab servers <https://www.aiidalab.net/>`_.
1. Log into one of the `open AiiDAlab servers <https://www.aiidalab.net/deployments/>`_.

Pipfile Outdated
@@ -0,0 +1,11 @@
[[source]]
Copy link
Member

Choose a reason for hiding this comment

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

I don't think this file is needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry for that I will remove it

@Mahhheshh
Copy link
Contributor Author

Mahhheshh commented Mar 23, 2023

I guess I have messed up in hurry I will make another pull request sorry for in inconvenience, please close this one for now.

Mahhheshh and others added 2 commits March 23, 2023 23:14
This commit fixes the outdated links in `docs/source/conf.py` and `docs/source/usage/index.rst` that were pointing to the old AiiDAlab URL. The links have been updated to point to the new URL.
@yakutovicha yakutovicha merged commit deec085 into aiidalab:main Mar 23, 2023
@yakutovicha
Copy link
Member

I guess I have messed up in hurry I will make another pull request sorry for in inconvenience, please close this one for now.

Looks all good now, thanks @Mahhheshh 👍

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.

None yet

2 participants