New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(926): Display template namespace if it exists #299
Conversation
app/templates/index/route.js
Outdated
@@ -12,6 +12,12 @@ export default Route.extend({ | |||
templates.forEach((t) => { | |||
if (!names[t.name]) { | |||
names[t.name] = 1; | |||
// construct full template name | |||
t.fullName = t.name; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we define this in a common place, second time this logic is repeated.
@@ -1,8 +1,8 @@ | |||
<article class="template"> | |||
<header> | |||
{{#link-to "templates.detail" template.name}}<h4>{{template.name}}</h4>{{/link-to}} | |||
{{#link-to "templates.detail" template.fullName}}<h4>{{template.fullName}}</h4>{{/link-to}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
an extra space
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Context
Should display namespace/name if namespace exists and is not "default". Also showing namespace and name in the template detail page now.
Objective
This PR should fix it so it still shows namespace and name.
Example with namespace:
Example with default namespace:
Related links
Related to screwdriver-cd/screwdriver#926