Skip to content

Return only active timetables #39

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
Merged
kosyfrances merged 1 commit into from
Oct 4, 2017
Merged

Return only active timetables #39

kosyfrances merged 1 commit into from
Oct 4, 2017

Conversation

@kosyfrances
Copy link
Member

@kosyfrances kosyfrances commented Oct 3, 2017

#36

This PR ensures that only active timetables are returned from the API

@kosyfrances kosyfrances self-assigned this Oct 3, 2017
@kosyfrances kosyfrances requested review from kola-er and nbyodai October 3, 2017 19:02
@coveralls
Copy link

coveralls commented Oct 3, 2017

Coverage Status

Coverage remained the same at 88.479% when pulling 946f87e on 36__inactive_timetables into 2ef9f5e on staging.

@kosyfrances kosyfrances force-pushed the 36__inactive_timetables branch from 946f87e to a47734d Compare October 4, 2017 02:47
@coveralls
Copy link

coveralls commented Oct 4, 2017

Coverage Status

Coverage remained the same at 88.479% when pulling a47734d on 36__inactive_timetables into ade2ff7 on staging.

@kosyfrances kosyfrances merged commit 089bc56 into staging Oct 4, 2017
@kosyfrances kosyfrances deleted the 36__inactive_timetables branch October 4, 2017 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kola-er kola-er kola-er approved these changes

@nbyodai nbyodai Awaiting requested review from nbyodai

Assignees

@kosyfrances kosyfrances

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@kosyfrances @coveralls @kola-er