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

✨ Set download endpoints to use http in dev mode #592

Merged
merged 1 commit into from
Mar 11, 2021

Conversation

gsantia
Copy link
Contributor

@gsantia gsantia commented Mar 2, 2021

Allow developers running the study creator locally to test and use the /download endpoint. This is done by checking
if the DEVELOP setting is True. If so, return http:// instead of https:// in the download URLs.

Closes #585.

@gsantia gsantia added the feature New functionality label Mar 2, 2021
@gsantia gsantia self-assigned this Mar 2, 2021
creator/files/nodes/file.py Outdated Show resolved Hide resolved
creator/jobs/schema.py Outdated Show resolved Hide resolved
docker-compose.dataservice.yml Outdated Show resolved Hide resolved
@dankolbman
Copy link
Contributor

Can you add a description to the PR?

@gsantia gsantia force-pushed the download-endpoints branch 3 times, most recently from 140297c to abb8547 Compare March 3, 2021 17:19
@gsantia gsantia changed the title ✨🚧 Set download endpoints to use http in dev mode ✨ Set download endpoints to use http in dev mode Mar 9, 2021
@gsantia gsantia marked this pull request as ready for review March 9, 2021 21:37
@codecov
Copy link

codecov bot commented Mar 9, 2021

Codecov Report

Merging #592 (2b45747) into master (c4a803a) will increase coverage by 0.13%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #592      +/-   ##
==========================================
+ Coverage   92.59%   92.73%   +0.13%     
==========================================
  Files         108      108              
  Lines        4795     4801       +6     
==========================================
+ Hits         4440     4452      +12     
+ Misses        355      349       -6     
Impacted Files Coverage Δ
creator/files/nodes/file.py 100.00% <100.00%> (ø)
creator/files/nodes/version.py 96.29% <100.00%> (+0.29%) ⬆️
creator/jobs/schema.py 73.84% <100.00%> (+7.17%) ⬆️
creator/jobs/models.py 89.79% <0.00%> (+4.08%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c4a803a...2b45747. Read the comment docs.

Copy link
Contributor

@dankolbman dankolbman left a comment

Choose a reason for hiding this comment

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

Can you add another test to test the job log downoads? It should be pretty similar.

tests/files/test_downloads.py Outdated Show resolved Hide resolved
@gsantia gsantia requested a review from dankolbman March 10, 2021 22:48
@gsantia gsantia force-pushed the download-endpoints branch 2 times, most recently from a912211 to 033d390 Compare March 10, 2021 23:00
Copy link
Contributor

@dankolbman dankolbman left a comment

Choose a reason for hiding this comment

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

🎢

Nice! Been looking forward to this so I don't need to modify the urls whenever I'm testing locally.

Copy link
Contributor

@dankolbman dankolbman left a comment

Choose a reason for hiding this comment

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

🎢

Nice! Been looking forward to this so I don't need to modify the urls whenever I'm testing locally.

@gsantia gsantia merged commit 69b1d80 into master Mar 11, 2021
@gsantia gsantia deleted the download-endpoints branch March 11, 2021 18:29
dankolbman added a commit that referenced this pull request Apr 7, 2021
## Release 1.16.0

### Summary

- Emojis: 🔥 x1, ⬆️ x1, ✨ x2
- Categories: Removals x1, Ops x1, Additions x2

### New features and changes

- [#610](#610) - 🔥 Remove lingering ego auth fields - [f091a41](f091a41) by [dankolbman](https://github.com/dankolbman)
- [#606](#606) - ⬆️ Bump django from 2.2.13 to 2.2.18 - [8ef82bd](8ef82bd) by [dependabot[bot]](https://github.com/apps/dependabot)
- [#601](#601) - ✨ Make events in data_review node sortable with EventFilter - [48d03a3](48d03a3) by [XuTheBunny](https://github.com/XuTheBunny)
- [#592](#592) - ✨ Set download endpoints to use http in dev mode - [69b1d80](69b1d80) by [gsantia](https://github.com/gsantia)
@dankolbman dankolbman mentioned this pull request Apr 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Download endpoints should use http in dev mode
2 participants