-
Notifications
You must be signed in to change notification settings - Fork 7
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
POWERDev page from old site #135
Conversation
Ref: d7.osuosl.org/services/powerdev
@knightsamar this PR should contain the submodule update from osuosl/dougfir-pelican-theme#77 once that's merged. |
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 recommend we go ahead and merge this as-is w/o the css table update. It's more important to get this back on the site than to make sure it looks 100% the same as before.
@knightsamar can you please also create a new PR which includes any missing projects? I wasn't sure if this included any new projects.
POWER Projects Tab
@ramereth , could you please review the pruned project and academic partner list? Sorry, I noticed your comment after pushing. I can move the commits to another PR if required. This list is now created by cross-referencing IBM's spreadsheet with I also propose that we have a page/list for projects which were hosted before with us but now have moved on because of their success and/or requirements for more hardware like GATK, GLIBC. |
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.
@knightsamar there seems to be an issue with the CSV data in powerdev_open_source_projects.csv.
/home/lance/git/osl/osuosl-pelican/content/services/services_powerdev.rst:46: (ERROR/3) Error with CSV data in "csv-table" directive:
',' expected after '"'
d7b0c27
to
bcf0efd
Compare
@ramereth thanks for catching that! fixed and rebased. |
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.
LGTM
I agree that we should make a separate PR that lists "Past projects" so we show the history.
!bump patch ~ |
Works towards resolving #128 by adding CSV-powered tables containing list of POWERDev related projects