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: pretty search engine names #189

Closed
wants to merge 7 commits into from
Closed

feat: pretty search engine names #189

wants to merge 7 commits into from

Conversation

k4lizen
Copy link
Member

@k4lizen k4lizen commented Jan 26, 2024

  • added pretty names array
  • using pretty names in climode search results output
    • where else should these be used? engines.Info?

@k4lizen k4lizen changed the base branch from main to names January 26, 2024 18:23
@k4lizen k4lizen changed the title Pretty names feat: pretty search engine names Jan 26, 2024
@aleksasiriski
Copy link
Member

aleksasiriski commented Feb 6, 2024

This can be used when returning the JSON results via API, so that pretty names get rendered on the frontend. But I want this moved to comments next to where the engines' names are defined and autogenerated with enumer.go

Base automatically changed from names to main February 7, 2024 23:50
@aleksasiriski aleksasiriski mentioned this pull request Mar 13, 2024
6 tasks
}
}

func init() {
Copy link
Member

@aleksasiriski aleksasiriski Mar 13, 2024

Choose a reason for hiding this comment

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

This is a runtime error, we must check for it before building our images otherwise it could result in broken version being released. I managed to do this in my PR that tries to add pretty names.

@aleksasiriski
Copy link
Member

This should be closed in favor of #257

@aleksasiriski aleksasiriski deleted the pretty-names branch May 26, 2024 12:59
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.

2 participants