Skip to content
This repository was archived by the owner on May 22, 2021. It is now read-only.

expired ids should reject instead of returning null#311

Merged
dannycoates merged 3 commits intomasterfrom
expiredDownloadPage
Jul 25, 2017
Merged

expired ids should reject instead of returning null#311
dannycoates merged 3 commits intomasterfrom
expiredDownloadPage

Conversation

@dannycoates
Copy link
Copy Markdown
Contributor

@dannycoates dannycoates commented Jul 25, 2017

addresses the null problem in #309

@abhinadduri
Copy link
Copy Markdown
Collaborator

Looks good!

@ericawright
Copy link
Copy Markdown
Contributor

I think all of the odd behaviour in #309 and #159 have been fixed now!

@dannycoates dannycoates force-pushed the expiredDownloadPage branch from 9de0825 to dd0ab71 Compare July 25, 2017 19:21
@dannycoates dannycoates force-pushed the expiredDownloadPage branch from 32fbfd6 to 424eb2c Compare July 25, 2017 20:00
@dannycoates dannycoates merged commit 65c2499 into master Jul 25, 2017
@pdehaan pdehaan deleted the expiredDownloadPage branch July 31, 2017 05:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants