Merge lp:~gary-lasker/software-center/temp-disable-lobby-recommendations-view into lp:software-center

Proposed by Gary Lasker
Status: Merged
Merged at revision: 2764
Proposed branch: lp:~gary-lasker/software-center/temp-disable-lobby-recommendations-view
Merge into: lp:software-center
Diff against target: 110 lines (+26/-15)
4 files modified
debian/changelog (+5/-1)
softwarecenter/ui/gtk3/app.py (+10/-8)
softwarecenter/ui/gtk3/views/catview_gtk.py (+4/-2)
test/gtk3/test_catview.py (+7/-4)
To merge this branch: bzr merge lp:~gary-lasker/software-center/temp-disable-lobby-recommendations-view
Reviewer Review Type Date Requested Status
software-store-developers Pending
Review via email: mp+93268@code.launchpad.net

Description of the change

(resumit with shorter branch name, changelog was not pretty at all! d'oh!)

In preparation for FF release today, this branch temporarily hides the lobby recommendations opt-in and display panel pending depoyment of support for this feature on the production server. Once that has been deployed, we can back out this change (FIXMEs everywhere to document this).

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2012-02-15 17:39:18 +0000
+++ debian/changelog 2012-02-15 18:19:17 +0000
@@ -60,12 +60,16 @@
60 - implement the recommendations opt-in UI and flow, upload60 - implement the recommendations opt-in UI and flow, upload
61 of the user profile data after opt-in, and display of61 of the user profile data after opt-in, and display of
62 per-application recommendations in the details view62 per-application recommendations in the details view
63 * lp:~gary-lasker/software-center/temp-disable-lobby-recommendations-view:
64 - temporarily hide the lobby recommendations opt-in and display
65 view pending rollout of the corresponding support on the
66 production server
63 67
64 [ Juhana Jauhiainen ]68 [ Juhana Jauhiainen ]
65 * softwarecenter/backend/launchpad.py:69 * softwarecenter/backend/launchpad.py:
66 - replaced EDGE_SERVICE_ROOT with LPNET_SERVICE_ROOT. (LP: #737697)70 - replaced EDGE_SERVICE_ROOT with LPNET_SERVICE_ROOT. (LP: #737697)
6771
68 -- Gary Lasker <gary.lasker@canonical.com> Wed, 15 Feb 2012 12:37:53 -050072 -- Gary Lasker <gary.lasker@canonical.com> Wed, 15 Feb 2012 13:09:24 -0500
6973
70software-center (5.1.8) precise; urgency=low74software-center (5.1.8) precise; urgency=low
7175
7276
=== modified file 'softwarecenter/ui/gtk3/app.py'
--- softwarecenter/ui/gtk3/app.py 2012-02-15 16:55:32 +0000
+++ softwarecenter/ui/gtk3/app.py 2012-02-15 18:19:17 +0000
@@ -454,14 +454,16 @@
454454
455 def on_available_pane_created(self, widget):455 def on_available_pane_created(self, widget):
456 self.available_pane.searchentry.grab_focus()456 self.available_pane.searchentry.grab_focus()
457 # connect a signal to monitor the recommendations opt-in state and457 # FIXME: temporarily remove this with the other lobby recommendations
458 # persist the recommendations uuid on an opt-in458 # pieces and re-enable when the server-side support is in place
459 self.available_pane.cat_view.recommended_for_you_panel.connect(459# # connect a signal to monitor the recommendations opt-in state and
460 "recommendations-opt-in",460# # persist the recommendations uuid on an opt-in
461 self._on_recommendations_opt_in)461# self.available_pane.cat_view.recommended_for_you_panel.connect(
462 self.available_pane.cat_view.recommended_for_you_panel.connect(462# "recommendations-opt-in",
463 "recommendations-opt-out",463# self._on_recommendations_opt_in)
464 self._on_recommendations_opt_out)464# self.available_pane.cat_view.recommended_for_you_panel.connect(
465# "recommendations-opt-out",
466# self._on_recommendations_opt_out)
465 467
466 #~ def on_installed_pane_created(self, widget):468 #~ def on_installed_pane_created(self, widget):
467 #~ pass469 #~ pass
468470
=== modified file 'softwarecenter/ui/gtk3/views/catview_gtk.py'
--- softwarecenter/ui/gtk3/views/catview_gtk.py 2012-02-15 11:02:29 +0000
+++ softwarecenter/ui/gtk3/views/catview_gtk.py 2012-02-15 18:19:17 +0000
@@ -252,7 +252,8 @@
252252
253 self._append_whats_new()253 self._append_whats_new()
254 self._append_top_rated()254 self._append_top_rated()
255 self._append_recommended_for_you()255 # FIXME: Temporarily hide this as server support is not yet in place!!
256# self._append_recommended_for_you()
256 self._append_appcount()257 self._append_appcount()
257258
258 #self._append_video_clips()259 #self._append_video_clips()
@@ -485,7 +486,8 @@
485486
486 self._update_top_rated_content()487 self._update_top_rated_content()
487 self._update_whats_new_content()488 self._update_whats_new_content()
488 self._update_recommended_for_you_content()489 # FIXME: Temporarily remove this as server support is not yet in place!!
490# self._update_recommended_for_you_content()
489 self._update_appcount()491 self._update_appcount()
490 return492 return
491493
492494
=== modified file 'test/gtk3/test_catview.py'
--- test/gtk3/test_catview.py 2012-02-14 00:39:53 +0000
+++ test/gtk3/test_catview.py 2012-02-15 18:19:17 +0000
@@ -85,8 +85,9 @@
85 self.assertFalse(view.whats_new_frame.get_property("visible"))85 self.assertFalse(view.whats_new_frame.get_property("visible"))
86 self._p()86 self._p()
87 win.destroy()87 win.destroy()
88 88
89 def test_subcatview_recommended_for_you_opt_in_display(self):89 # FIXME: Temporarily disable this as server support is not yet in place!!
90 def disabled_test_subcatview_recommended_for_you_opt_in_display(self):
90 from softwarecenter.ui.gtk3.views.catview_gtk import get_test_window_catview91 from softwarecenter.ui.gtk3.views.catview_gtk import get_test_window_catview
91 # get the widgets we need92 # get the widgets we need
92 win = get_test_window_catview()93 win = get_test_window_catview()
@@ -101,7 +102,8 @@
101 # patch out the agent query method to avoid making the actual server call102 # patch out the agent query method to avoid making the actual server call
102 @patch('softwarecenter.backend.recagent.RecommenderAgent'103 @patch('softwarecenter.backend.recagent.RecommenderAgent'
103 '.query_submit_profile')104 '.query_submit_profile')
104 def test_subcatview_recommended_for_you_spinner_display(self, mock_query):105 # FIXME: Temporarily disable this as server support is not yet in place!!
106 def disabled_test_subcatview_recommended_for_you_spinner_display(self, mock_query):
105 from softwarecenter.ui.gtk3.views.catview_gtk import get_test_window_catview107 from softwarecenter.ui.gtk3.views.catview_gtk import get_test_window_catview
106 # get the widgets we need108 # get the widgets we need
107 win = get_test_window_catview()109 win = get_test_window_catview()
@@ -120,7 +122,8 @@
120 # patch out the agent query method to avoid making the actual server call122 # patch out the agent query method to avoid making the actual server call
121 @patch('softwarecenter.backend.recagent.RecommenderAgent'123 @patch('softwarecenter.backend.recagent.RecommenderAgent'
122 '.query_submit_profile')124 '.query_submit_profile')
123 def test_subcatview_recommended_for_you_display_recommendations(self, mock_query):125 # FIXME: Temporarily disable this as server support is not yet in place!!
126 def disabled_test_subcatview_recommended_for_you_display_recommendations(self, mock_query):
124 from softwarecenter.ui.gtk3.views.catview_gtk import get_test_window_catview127 from softwarecenter.ui.gtk3.views.catview_gtk import get_test_window_catview
125 # get the widgets we need128 # get the widgets we need
126 win = get_test_window_catview()129 win = get_test_window_catview()

Subscribers

People subscribed via source and target branches