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

Vojtch search engine #141

Closed
wants to merge 16 commits into from
Closed

Vojtch search engine #141

wants to merge 16 commits into from

Conversation

Vojtch159
Copy link
Collaborator

Přidal jsem možnost vyhledávání článku.

Search je zatím lineární. Aktuálně to není problém, vzhledem k počtu článků. Každopádně dlouhodobě to určitě bude chtít implementaci nějakého algoritmu co bude mít lepší časovou komplexitu. (Bohužel v tomto jsem doopravdy ani ne začátečník, takže jsem nějaký algoritmus neimplementoval hned od začátku, protože žádný neznám)

@Vojtch159 Vojtch159 added the enhancement New feature or request label May 16, 2021
@Vojtch159 Vojtch159 added this to the Verze 2.0.0 milestone May 16, 2021
@Vojtch159 Vojtch159 self-assigned this May 16, 2021
@Vojtch159 Vojtch159 linked an issue May 16, 2021 that may be closed by this pull request
Copy link
Collaborator

@jan-miksik jan-miksik left a comment

Choose a reason for hiding this comment

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

Volání api s limit: 'all', bych se radši vyhnul. Takhle se v podstatě stáhnou všechny články. Až tam bude hodně článků mohl by to být problém. Mělo by stačit vytvořit jednu stránku a na té volat to hledání, třeba přes query paramater. Přijde mi že getSearchPost se volá na všech stránkách.
Pravděpodobně bych zkusil některý z návrhů, které mají v ghost dokumentaci https://ghost.org/docs/search/

@Vojtch159
Copy link
Collaborator Author

Koukal jsem na to a myslím že se tomu volání api limit: all moc jednoduše nevyhneme.
Ty řešení co tam oni mají jsou:
a) Něco od googlu, co jsem úplně nepochopil jak funguje, ale vypadá to hrozně a nedá se to moc customizovat + posílá to ty data kdo ví kam, takže to nebude moc trustworthy.
b) Doporučují ten ghostHunter framework. Každopádně to je de facto totéž co jsem napsal já, jen je to jako bundle a má to víc funkcí, které nepotřebujeme.
c) Doporučují tam na to nějakou knihovnu, ale když klinu na odkaz tak mě to hodí na github 404 not found....

ad. b) To volání api s limit: all je tady
https://github.com/jamalneufeld/ghostHunter/blob/42af31a715d497ee198b42918675efcc51f513a9/src/ghosthunter.js
řádek 253.

Jediné co mě napadá, tak je to, že bych napsal vlastní backend, který by ty články jednou za pár sekund bral k nám na server a udělal by se rest api endpoint, který by vracel array názvů všech článků. A potom už si ty kompletní články vezmou z toho ghost api jen jaké potřebujeme.

@Vojtch159 Vojtch159 added the bug Something isn't working label Jun 29, 2021
@Vojtch159
Copy link
Collaborator Author

Ruším pull request než search engine dodělám s novými věcmi.

@Vojtch159 Vojtch159 closed this Jul 10, 2021
@Vojtch159 Vojtch159 deleted the Vojtch-SearchEngine branch July 11, 2021 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

2 participants