Skip to content

Renaming gem to prexian - #88

Open
ronaldtse wants to merge 31 commits into
mainfrom
rt-prexus
Open

Renaming gem to prexian#88
ronaldtse wants to merge 31 commits into
mainfrom
rt-prexus

Conversation

@ronaldtse

Copy link
Copy Markdown
Contributor

No description provided.

@ronaldtse ronaldtse changed the title Renaming gem to prexus Renaming gem to prexian May 26, 2025
@ronaldtse

ronaldtse commented May 27, 2025

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
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
Copy Markdown
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