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
1=== modified file 'debian/changelog'
2--- debian/changelog 2012-02-15 17:39:18 +0000
3+++ debian/changelog 2012-02-15 18:19:17 +0000
4@@ -60,12 +60,16 @@
5 - implement the recommendations opt-in UI and flow, upload
6 of the user profile data after opt-in, and display of
7 per-application recommendations in the details view
8+ * lp:~gary-lasker/software-center/temp-disable-lobby-recommendations-view:
9+ - temporarily hide the lobby recommendations opt-in and display
10+ view pending rollout of the corresponding support on the
11+ production server
12
13 [ Juhana Jauhiainen ]
14 * softwarecenter/backend/launchpad.py:
15 - replaced EDGE_SERVICE_ROOT with LPNET_SERVICE_ROOT. (LP: #737697)
16
17- -- Gary Lasker <gary.lasker@canonical.com> Wed, 15 Feb 2012 12:37:53 -0500
18+ -- Gary Lasker <gary.lasker@canonical.com> Wed, 15 Feb 2012 13:09:24 -0500
19
20 software-center (5.1.8) precise; urgency=low
21
22
23=== modified file 'softwarecenter/ui/gtk3/app.py'
24--- softwarecenter/ui/gtk3/app.py 2012-02-15 16:55:32 +0000
25+++ softwarecenter/ui/gtk3/app.py 2012-02-15 18:19:17 +0000
26@@ -454,14 +454,16 @@
27
28 def on_available_pane_created(self, widget):
29 self.available_pane.searchentry.grab_focus()
30- # connect a signal to monitor the recommendations opt-in state and
31- # persist the recommendations uuid on an opt-in
32- self.available_pane.cat_view.recommended_for_you_panel.connect(
33- "recommendations-opt-in",
34- self._on_recommendations_opt_in)
35- self.available_pane.cat_view.recommended_for_you_panel.connect(
36- "recommendations-opt-out",
37- self._on_recommendations_opt_out)
38+ # FIXME: temporarily remove this with the other lobby recommendations
39+ # pieces and re-enable when the server-side support is in place
40+# # connect a signal to monitor the recommendations opt-in state and
41+# # persist the recommendations uuid on an opt-in
42+# self.available_pane.cat_view.recommended_for_you_panel.connect(
43+# "recommendations-opt-in",
44+# self._on_recommendations_opt_in)
45+# self.available_pane.cat_view.recommended_for_you_panel.connect(
46+# "recommendations-opt-out",
47+# self._on_recommendations_opt_out)
48
49 #~ def on_installed_pane_created(self, widget):
50 #~ pass
51
52=== modified file 'softwarecenter/ui/gtk3/views/catview_gtk.py'
53--- softwarecenter/ui/gtk3/views/catview_gtk.py 2012-02-15 11:02:29 +0000
54+++ softwarecenter/ui/gtk3/views/catview_gtk.py 2012-02-15 18:19:17 +0000
55@@ -252,7 +252,8 @@
56
57 self._append_whats_new()
58 self._append_top_rated()
59- self._append_recommended_for_you()
60+ # FIXME: Temporarily hide this as server support is not yet in place!!
61+# self._append_recommended_for_you()
62 self._append_appcount()
63
64 #self._append_video_clips()
65@@ -485,7 +486,8 @@
66
67 self._update_top_rated_content()
68 self._update_whats_new_content()
69- self._update_recommended_for_you_content()
70+ # FIXME: Temporarily remove this as server support is not yet in place!!
71+# self._update_recommended_for_you_content()
72 self._update_appcount()
73 return
74
75
76=== modified file 'test/gtk3/test_catview.py'
77--- test/gtk3/test_catview.py 2012-02-14 00:39:53 +0000
78+++ test/gtk3/test_catview.py 2012-02-15 18:19:17 +0000
79@@ -85,8 +85,9 @@
80 self.assertFalse(view.whats_new_frame.get_property("visible"))
81 self._p()
82 win.destroy()
83-
84- def test_subcatview_recommended_for_you_opt_in_display(self):
85+
86+ # FIXME: Temporarily disable this as server support is not yet in place!!
87+ def disabled_test_subcatview_recommended_for_you_opt_in_display(self):
88 from softwarecenter.ui.gtk3.views.catview_gtk import get_test_window_catview
89 # get the widgets we need
90 win = get_test_window_catview()
91@@ -101,7 +102,8 @@
92 # patch out the agent query method to avoid making the actual server call
93 @patch('softwarecenter.backend.recagent.RecommenderAgent'
94 '.query_submit_profile')
95- def test_subcatview_recommended_for_you_spinner_display(self, mock_query):
96+ # FIXME: Temporarily disable this as server support is not yet in place!!
97+ def disabled_test_subcatview_recommended_for_you_spinner_display(self, mock_query):
98 from softwarecenter.ui.gtk3.views.catview_gtk import get_test_window_catview
99 # get the widgets we need
100 win = get_test_window_catview()
101@@ -120,7 +122,8 @@
102 # patch out the agent query method to avoid making the actual server call
103 @patch('softwarecenter.backend.recagent.RecommenderAgent'
104 '.query_submit_profile')
105- def test_subcatview_recommended_for_you_display_recommendations(self, mock_query):
106+ # FIXME: Temporarily disable this as server support is not yet in place!!
107+ def disabled_test_subcatview_recommended_for_you_display_recommendations(self, mock_query):
108 from softwarecenter.ui.gtk3.views.catview_gtk import get_test_window_catview
109 # get the widgets we need
110 win = get_test_window_catview()

Subscribers

People subscribed via source and target branches