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.

Sign up for GitHub

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

Jump to bottom

fix: Show only upcoming or ongoing events #1753

Merged
merged 2 commits into from May 17, 2019
Merged

fix: Show only upcoming or ongoing events #1753

merged 2 commits into from May 17, 2019

Conversation

liveHarshit
Copy link
Member

Fixes #1748

@auto-label auto-label bot added the fix label May 11, 2019
@iamareebjamal
Copy link
Member

Apply server-side filter for this

@liveHarshit
Copy link
Member Author

liveHarshit commented May 12, 2019 •
edited

How filter is being used in the server. It is showing a blank box in the example.
image

Apply server-side filter for this

@nikit19
Copy link
Member

nikit19 commented May 12, 2019 •
edited by unfurl-links bot

@liveHarshit https://flask-restless.readthedocs.io/en/latest/filtering.html

@liveHarshit
Copy link
Member Author

liveHarshit commented May 15, 2019 •
edited

It should like that {"name":"ends-at","op":">","val":"%${System.currentTimeMillis()}%"}, Here ends-at need to convert in mili seconds. But before query cannot retrieve the event.

@iamareebjamal
Copy link
Member

See date range queries on the frontend.

@uds5501
Copy link

uds5501 commented May 15, 2019 •
edited by unfurl-links bot

@liveHarshit This filter is the one used in Upcoming events in frontend, let me know if you require any assistance

https://github.com/fossasia/open-event-frontend/blob/125f26b44bd8f9a98710959ddae70c76068284f4/app/routes/index.js#L13-L123

GitHub
The frontend for the Open Event API Server https://eventyay.com | https://open-event-fe.netlify.com - fossasia/open-event-frontend

@liveHarshit
Copy link
Member Author

Updated

@iamareebjamal iamareebjamal merged commit d6b7ead into fossasia:development May 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nikit19 nikit19 nikit19 approved these changes

Assignees
No one assigned
Labels
fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Show only upcoming or ongoing events
4 participants
@liveHarshit @iamareebjamal @nikit19 @uds5501