Skip to content

Renaming gem to prexian #88

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

Open
wants to merge 30 commits into
base: main
Choose a base branch
from
Open

Renaming gem to prexian #88

wants to merge 30 commits into from

Conversation

ronaldtse
Copy link
Contributor

No description provided.

@ronaldtse ronaldtse changed the title Renaming gem to prexus Renaming gem to prexian May 26, 2025
@ronaldtse
Copy link
Contributor Author

ronaldtse commented May 27, 2025

There are a few bugs remaining:

Issues in the hub site

Steps:

script/test-hub
  1. The project blog posts are showing on the front page but are not rendering individually.
    Then scroll to the bottom:
Screenshot 2025-05-27 at 9 47 55 AM

The left blog post is from the project, the right blog post is from the hub. The one on the left is not available in rendered form if you click on it. The one on the right renders fine.

Screenshot 2025-05-27 at 9 48 51 AM
  1. The blog index is not working
Screenshot 2025-05-27 at 10 22 12 AM
  1. The projects index is also not working even though there are clearly projects loaded
Screenshot 2025-05-27 at 10 22 25 AM

Issues in the project site

script/test-project

http://127.0.0.1:4000
Screenshot 2025-05-27 at 10 17 20 AM

  1. The blog index is showing up as blank, but the blog post is viewable.
  2. The blog post rectangle is also not showing the white background
  3. The hub logo is missing in the footer.

Approach

Jekyll uses site.static_files for its static assets. We should make the hub/assets/* files available there so we can test for their existence instead of needing to include them.

@ronaldtse
Copy link
Contributor Author

The final test is to apply this gem to this pair of sites:

@ronaldtse ronaldtse mentioned this pull request May 27, 2025
@ronaldtse
Copy link
Contributor Author

For @hasannadeem . Thanks!

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.

1 participant