Skip to content
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(search): add snapshoted latest version and date of a package #1410

Closed
wants to merge 1 commit into from

Conversation

94noni
Copy link
Contributor

@94noni 94noni commented Jan 8, 2024

Hello,

Small PR related to issues mentioned/related.

Note

PR opened as is to gather feedbacks/have raw material for discussions (not finished at all)
I am open to discuss the easiest/fastest/simplest way for introducing such data that will help a think a lot the search results


Idea of display

Capture d’écran 2024-01-08 à 18 47 42

refs:

src/Entity/Package.php Outdated Show resolved Hide resolved
src/Entity/Package.php Outdated Show resolved Hide resolved
@@ -174,6 +174,8 @@ search.addWidget(
<div class="col-sm-3 col-lg-2">
{{#meta}}
<p class="metadata">
<span class="metadata-block"><i class="glyphicon glyphicon-tags"></i> {{ meta.latest_version }}</span>
<span class="metadata-block"><i class="glyphicon glyphicon-calendar"></i> {{ meta.latest_date }}</span>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what if those are not available ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i have not install/run the project locally for now, dont want to invest much time of PR is not accepted
so for now, it was just to make a decent screenshot with fake data ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants