-
Notifications
You must be signed in to change notification settings - Fork 60
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
FEAT: Community package listing pages for astropy + SunPy #207
Conversation
_data/packages.yml
Outdated
@@ -1,3 +1,90 @@ | |||
- package_name: astro-package3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these are just dummy placeholder packages to test the search for astropy functionality - do not worry about the information in each of these but if we need new FIELDS that is something to consider
@lwasser , I vaguely remember you said you added a link back to "original review" to the listing of an accepted package in a mock-up. Was it this mockup? I don't see a preview link and I am not sure if the screenshots are up-to-date. Hope you can clarify. Thanks! |
hey @pllim i've actually been working on this!! it's not in this mockup branch YET (i need to rebase) BUT you can see it live on the website - i just pushed it. it took me a bit of time as i had to update the code that parses our reviews. but it's done now!! check it our here . now that it's working you can imaging the exact same thing on this mockup branch once i rebase! There is a tiny bug i need to fix on the packaging page in terms of sorting but that will be fixed in the next 30 minutes :) let me know if you have any other questions or if you need anything else. I have it on my list to review the APE in the upcoming days (most likely on monday). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have a lot of comments when it comes to website designs. I should ping those listed under "Astropy.org web page maintainer" on https://www.astropy.org/team : @hamogu @dhomeier @eteq
At a glance though, you might need Astropy logo and banner that are also compatible with dark mode. We have some here that we use for astropy
documentation: https://github.com/astropy/astropy/tree/main/docs/_static
Thanks!
@lwasser Do we still want this open or should it be closed? |
Closed? I thought Astropy had agreement with pyOpenSci on cross-listing? Will this be replaced with something else? |
oh this definitely needs to be reviewed and merged!! We don't want to close it. this just fell by the wayside with the fall festival and other items going on. I am actually onboarding someone now to help with these types of tasks so they don't go stale because of me!! i'll make sure this one is a priority, but it may be another month until we get up and running with this! |
@pllim maybe we just merge it as is (with someone fixing whatever is buggy in the pr) and allow for the astropy community to chime in as they see issues in the future? we have a handful of astropy packages now, so the page should look better than when we started with no packages!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not a javascript programmer, so I can't say much about the implementation, but I still think that we want a page like that - and with the number of accepted packages going up, now is a good time to do so. I suggest to merge this relatively quickly and if we want to iterate on the design (I don't have any strong opinions on that, though) then we can always do that in follow-up PRs.
@hamogu, this sounds good to me. |
_pages/python-packages.md
Outdated
|
||
To view packages affiliated with our partner communities that are a part of our peer review partnership program, click on a community link below. [Learn more about our community partnerships here.](#) | ||
|
||
<a href="/partners/astropy.html"><button class="button community">Astropy</button></a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this link be href="/communities/astropy.html"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh i like using the word community in the url over partners. yes let's update it. we will want to update the yaml as well for the page. I will do that locally!!
<li><a href="{{ apackage.archive }}" rel="permalink"><i class="fas fa-bookmark fa-fw"></i> JOSS Approved</a></li> | ||
{% endif %} | ||
{% if apackage.astropy %} | ||
<li><a href="partners/astropy.html"><i class="fa-solid fa-check-double"></i> <img src="http://img.shields.io/badge/Affiliated-Astropy-orange.svg?style=flat" alt="Astropy" /></a></li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we using partners or communities in this link?
<li><a href="partners/astropy.html"><i class="fa-solid fa-check-double"></i> <img src="http://img.shields.io/badge/Affiliated-Astropy-orange.svg?style=flat" alt="Astropy" /></a></li> | ||
{% endif %} | ||
{% if apackage.sunpy %} | ||
<li><a href="partners/astropy.html"><i class="fa-solid fa-check-double"></i> <img src="http://img.shields.io/badge/Affiliated-Sunpy-yellow.svg?style=flat" alt="Astropy" /></a></li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same question here.
communities/astropy.md
Outdated
overlay_filter: 0.6 | ||
overlay_image: images/partners/astropy-banner.png | ||
image_alt: "Digital elevation model" | ||
permalink: partners/astropy.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
communities or partners
Ok friends. I am going to merge this with the updates. The new page shows 3 Astropy packages have been accepted! 🚀 |
the ci failures
|
FWIW I opened a couple of follow-up issues. Thanks again, all! |
wonderful. thanks @pllim |
Landing page with community partner buttons
First - the landing page with new community specific buttons at the top leading to a community partner page
Search feature to find astropy affiliated packages on our main python-packages landing page.
Then we have the landing page where someone might search for astropy to find related packages.
Community landing page
I envision something like this (styles can be fixed) that listed only the affiliated packages. A few notes