Skip to content

add links to issues in overview#39

Merged
ekohl merged 1 commit intomasterfrom
issues
Feb 12, 2026
Merged

add links to issues in overview#39
ekohl merged 1 commit intomasterfrom
issues

Conversation

@evgeni
Copy link
Copy Markdown
Member

@evgeni evgeni commented Sep 15, 2025

No description provided.

@ekohl ekohl merged commit daeb52a into master Feb 12, 2026
3 checks passed
@ekohl ekohl deleted the issues branch February 12, 2026 09:23
markdown_link('RPM', entry.rpm_url),
markdown_link('deb', entry.deb_url),
markdown_links(entry.test_urls),
markdown_link('Issues', entry.issues_url),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After merging this I looked at the result and wondered if we should say where it points to: GH or Redmine. Not sure if it helps, but noting it down here.

Copy link
Copy Markdown
Member Author

@evgeni evgeni Feb 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you mean something like

Suggested change
markdown_link('Issues', entry.issues_url),
markdown_link('Redmine' if entry.redmine else 'GitHub', entry.issues_url),

or even further

def issues_link(issues_url):
  if 'github.com' in issues_url:
    issues_label = 'GitHub'
  elif 'projects.theforeman.org' in issues_url:
    issues_label = 'Redmine'
  else:
    issues_label = 'Issues'
  return markdown_link(issues_label, issues_url)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#44

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants