[OCPERT-108] Make advisory urls clickable in the test-report #525
Conversation
openshift-ci
bot
added
the
do-not-merge/work-in-progress
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
jhuttana
changed the title
openshift-ci
bot
removed
the
do-not-merge/work-in-progress
| @@ -160,7 +160,7 @@ def update_advisory_info(self, ad): | |||
| Args: | |||
| ad (str): advisories of current release | |||
| """ | |||
| self._ws.update_acell(LABEL_ADVISORY, ad) | |||
| self._to_hyperlink(self._ws.update_acell(LABEL_ADVISORY, ad)) | |||
There was a problem hiding this comment.
Does it create hyperlink to all advisories or just one hyperlink?
There was a problem hiding this comment.
I think we want to add links for all the advisories in this cell, this method does not work. =HYPERLINK does not support multiple links in one cell. we need to use userEnteredValue in batch update request.
xref: https://stackoverflow.com/questions/77310503/how-to-add-multiple-hyperlinks-in-a-single-cell-in-google-sheets-using-python
There was a problem hiding this comment.
+1
I found the same:
- Adding hyperlinks by formula is limited to only one.
- Multiple links can be added manually by
Ctrl/Cmd + Kor by the api script.
JIRA: https://issues.redhat.com/browse/OCPERT-108
Could you please take a look?