Conversation
Empty charmbrowser container on destroy Test added.
var cachedResults = this.get('cachedResults')[apiEndpoint]; | ||
if (cachedResults) { | ||
callbacks.success(cachedResults); | ||
return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for clarifying that you don't need to cancel in flight requests here - Can you add a comment to why this return is required for future us :)
👍 with some requests for comments. QA'ing now |
Couple issues with QA:
|
Test PASSed. |
Deferring QA issues to a separate branch. |
Status: merge request accepted. Url: http://ci.jujugui.org:8080/job/juju-gui-merge |
Test PASSed. |
This is a first iteration of caching charmbrowser requests. To QA, with il flag: * deploy a service * closer the inspector; interesting should load immediately * search for apache * deploy a service * close the inspector; apache search results should load imeediately.
Test FAILed. |
This is a first iteration of caching charmbrowser requests.
To QA, with il flag: