openstack/releases
Code Issues Proposed changes

Fix Releases Dashboard link

Browse Source 
The releases dashboard link does not work properly recently. It seems
that there are two problem with the link:
* the link does not need the '#' anymore
* due to some reason the 'All Releases' query messes up the dashboard
  (at first place it does not need the 'is:open' part because it's
  added already with 'foreach', secondly it gives an error message:
  'Error 400 (Bad Request): limit of 10 queries'). Nevertheless, 'All
  Releases' is probably not that important as all the release patches
  are covered with series sections together with 'Others' section
  (except deliverables under 'deliverables/<series_in_EM_or_EOL>/*').
  So this patch fixes the dashboard by removing 'All Releases' section.

Note: This problem comes even under Firefox or Chrome (when signed in).

Change-Id: I9226e7b7e9b9594daa11d97bd6a8b4b04c56f596
This commit is contained in:
Előd Illés 2022-05-02 19:21:52 +02:00
parent a9ebf39b89
commit c0ee70a74a
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
doc/source/_exts/reviewinbox.py
View File

@ -34,7 +34,7 @@ class ReviewInbox(rst.Directive):
has_content = True has_content = True
def run(self): def run(self):
url = 'https://review.opendev.org/#/dashboard/?' url = 'https://review.opendev.org/dashboard/?'
sections = [] sections = []
url += escape(urllib_parse.urlencode({ url += escape(urllib_parse.urlencode({
@ -70,9 +70,6 @@ class ReviewInbox(rst.Directive):
r'( project:openstack/releases file:^openstack_releases/.* ) OR ' r'( project:openstack/releases file:^openstack_releases/.* ) OR '
r'project:openstack/reno)'}))) r'project:openstack/reno)'})))
sections.append(escape(urllib_parse.urlencode({
'All Releases': 'is:open project:openstack/releases'})))
url += '&'.join(sections) url += '&'.join(sections)
result = sm.ViewList() result = sm.ViewList()