Skip to content

Commit

Permalink
Merge pull request #298 from communitybridge/feature/lfx-header-v2
Browse files Browse the repository at this point in the history
Added missing element for LFX v2 header
  • Loading branch information
luismoriguerra authored Jan 26, 2023
2 parents a9304aa + e37e1f5 commit 51a6061
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 11 deletions.
15 changes: 5 additions & 10 deletions src/app/shared/services/lfx-header.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,21 +7,16 @@ import { EnvConfig } from '../../config/cla-env-utils';
import { AppSettings } from '../../config/app-settings';

const script = document.createElement('script');
script.setAttribute(
'src',
EnvConfig.default[AppSettings.LFX_HEADER]
);
script.setAttribute('src', EnvConfig.default[AppSettings.LFX_HEADER]);
document.head.appendChild(script);

@Injectable({
providedIn: 'root'
providedIn: 'root',
})
export class LfxHeaderService {
links: any[];

constructor(
private auth: AuthService
) {
constructor(private auth: AuthService) {
this.setUserInLFxHeader();
this.setLinks();
this.setCallBackUrl();
Expand All @@ -39,8 +34,8 @@ export class LfxHeaderService {
},
{
title: 'Developer',
url: AppSettings.LEARN_MORE
}
url: AppSettings.LEARN_MORE,
},
];
const element: any = document.getElementById('lfx-header-v2');
element.links = this.links;
Expand Down
2 changes: 1 addition & 1 deletion src/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
<body>
<lfx-header-v2
product="LF CLA"
id="lfx-header"
id="lfx-header-v2"
supportlink="https://jira.linuxfoundation.org/plugins/servlet/theme/portal/4/create/143"
docslink="https://docs.linuxfoundation.org/lfx/easycla/v2-current"
faqlink="https://docs.linuxfoundation.org/lfx/easycla/v2-current/getting-started/easycla-faqs"
Expand Down

0 comments on commit 51a6061

Please sign in to comment.