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

Řazení meetupů #86

Closed
frenzymadness opened this issue Mar 5, 2018 · 1 comment · Fixed by #87
Closed

Řazení meetupů #86

frenzymadness opened this issue Mar 5, 2018 · 1 comment · Fixed by #87

Comments

@frenzymadness
Copy link
Member

Pokud je naplánován více než jeden meetup dopředu, neřadí se správně a na mapce a v přehledu pod ní se nezobrazuje ten nejbližší. Jako například teď v Ostravě.

Kód, který by to měl zařídit tam ovšem je, takže tam bude nějaky bug. https://github.com/pyvec/pyvo.cz/blob/master/pyvocz/views.py#L95

image

encukou added a commit to encukou/pyvo.cz that referenced this issue Mar 5, 2018
Previously, the latest planned event was shown on the home page for
each series.
So, when Ostrava Pyvo was planned two months (two events) in advance,
the one in the far future would be shown.

Reuse the logic for ordering series to also apply for picking the event:
prefer a future one if available, then sort by distance from today.

Related to: pyvec#86
encukou added a commit to encukou/pyvo.cz that referenced this issue Mar 5, 2018
Previously, the latest planned event was shown on the home page for
each series.
So, when Ostrava Pyvo was planned two months (two events) in advance,
the one in the far future would be shown.

Reuse the logic for ordering series to also apply for picking the event:
prefer a future one if available, then sort by distance from today.

Related to: pyvec#86
@encukou encukou reopened this Mar 5, 2018
@encukou
Copy link
Member

encukou commented Mar 5, 2018

Stejný problém je na stránce města: https://pyvo.cz/ostrava-pyvo/ ukazuje na prvním místě sraz, který je nejdál v budoucnosti. A ten nedcházející je v "historii srazů".

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 a pull request may close this issue.

2 participants