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

Allow photos to be displayed in full screen #418

Merged
merged 9 commits into from
Jun 6, 2020
Merged

Conversation

danactive
Copy link
Owner

@danactive danactive commented Mar 22, 2020

Summary
This should decrease the number of Dropbox get_temporary_link XHR calls

Test plan
Yes unit tests moved from reducer to saga

@coveralls
Copy link

coveralls commented Mar 22, 2020

Coverage Status

Coverage remained the same at 93.056% when pulling fd60559 on danactive/reselect into ea368f6 on master.

@danactive danactive force-pushed the danactive/reselect branch from 879f051 to c8684ed Compare March 22, 2020 21:24
@danactive danactive force-pushed the danactive/reselect branch from c8684ed to 9dee9af Compare March 23, 2020 05:26
@danactive danactive force-pushed the danactive/reselect branch 3 times, most recently from 9d87651 to 71535e9 Compare April 1, 2020 06:05
@danactive danactive force-pushed the danactive/reselect branch from 71535e9 to 5d9ce5b Compare April 1, 2020 06:06
@danactive danactive force-pushed the danactive/reselect branch from 3f53176 to bbce59d Compare June 5, 2020 03:49
@danactive danactive force-pushed the danactive/reselect branch from bbce59d to c34d4cc Compare June 5, 2020 18:01
@danactive
Copy link
Owner Author

Close #38

@danactive danactive merged commit 3d32ed2 into master Jun 6, 2020
@danactive danactive deleted the danactive/reselect branch June 6, 2020 04:25
@danactive danactive changed the title Danactive/reselect Allow photos to be displayed in full screen Jun 6, 2020
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 this pull request may close these issues.

2 participants