Skip to content

Multiple improvements during recent PTS25 #72

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

Merged
merged 30 commits into from
May 4, 2025
Merged

Multiple improvements during recent PTS25 #72

merged 30 commits into from
May 4, 2025

Conversation

atoomic
Copy link
Member

@atoomic atoomic commented May 4, 2025

That pull request contains multiple improvements developed during PTS 2025

Improvements

  • switch development environment to docker compose
  • Update HomePage Design
  • Preserve filter in pagination
  • Update font-awesome to version 4.7.0
  • Update assets libraries
  • CDN cache more static pages by adding some http headers
  • Cache improvements
  • Add support for multiple search filter
  • Dynamic placeholders for input elements
  • Add some hint when using invalid search
  • Mount search repo read only to simulate production environment
  • Switch CI to GitHub action and use containers

Fixes:

  • Fix output parser to show more results
  • Fix pagination issue +/-10 pages
  • Fix ignore list in search
  • Remove link to distro when already on distro page

atoomic added 30 commits May 1, 2025 12:09
Fix #70

Preserve exluded filenames in pagination
That commit is allowing easier contribution by using
docker container to develop and release the application.

A Makefile is providing the basic wrapper to start the container.
Fix #69

The pattern to ignore files was not used in the cache key.
Restore it so we can use the accurate cache content.
Fix #63

The git output parser was pretty naive and buggy when multiple
chunks were found from the same file.

This was resulting in skipping results we should have displayed to begin with.
That changeset is fixing that issue and will now show more results.

The summary was accurate but the output truncated due to poor man parsing reason.
This is providing a better first time search
The list can be customized or restored anytime
by checking the box.
@atoomic atoomic requested a review from ssoriche May 4, 2025 07:46
@atoomic atoomic merged commit 0c302e6 into master May 4, 2025
2 checks passed
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