Skip to content
This repository has been archived by the owner on Jan 24, 2023. It is now read-only.

Add deployment info for apps deployed via docker & fix info for local/archive apps #3291

Merged
merged 4 commits into from
Jan 7, 2019

Conversation

richard-cox
Copy link
Contributor

…/archive apps

- Show deployment info for apps deployed via docker image
  - Show link to docker hub image when appropriate
- Improved dpeloyment info when loading (no flickering of title when going from no info --> have info)
- Fixed bug where apps deployed from file/archive threw exception and failed to show deploy info
- Icon needs updating with version from https://icons8.com/icon/set/docker/material
  - FontForge on linux isn't behaving
  - Manual c&p also failing
@cfdreddbot
Copy link

✅ Hey richard-cox! The commit authors and yourself have already signed the CLA.

@codecov
Copy link

codecov bot commented Dec 18, 2018

Codecov Report

Merging #3291 into v2-master will decrease coverage by 0.12%.
The diff coverage is 67.16%.

@@              Coverage Diff              @@
##           v2-master    #3291      +/-   ##
=============================================
- Coverage       70.9%   70.78%   -0.13%     
=============================================
  Files            642      642              
  Lines          28204    28369     +165     
  Branches        6412     6450      +38     
=============================================
+ Hits           19999    20081      +82     
- Misses          8205     8288      +83

@richard-cox richard-cox merged commit bae594b into v2-master Jan 7, 2019
@richard-cox richard-cox deleted the add-docker-deployment-info branch January 7, 2019 17:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants