admin/twenty
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fixed favicon requests for empty domain names (#4191)

Browse Source 
* Fixed favicon requests for empty domain names

* Fixed the test case for undefined domain name
This commit is contained in:
Sohal Kumar Singh
2024-02-27 13:01:51 +05:30
committed by GitHub
parent 8b39e53e49
commit 368edf70b5
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
packages/twenty-front/src/utils/__tests__/utils.test.ts
View File

@ -42,8 +42,6 @@ describe('getLogoUrlFromDomainName', () => {
}); });
test('should handle undefined input', () => { test('should handle undefined input', () => {
expect(getLogoUrlFromDomainName(undefined)).toBe( expect(getLogoUrlFromDomainName(undefined)).toBe(undefined);
'https://favicon.twenty.com/',
);
}); });
}); });

4
packages/twenty-front/src/utils/index.ts
View File

@ -20,5 +20,7 @@ export const getLogoUrlFromDomainName = (
domainName?: string, domainName?: string,
): string | undefined => { ): string | undefined => {
const sanitizedDomain = sanitizeURL(domainName); const sanitizedDomain = sanitizeURL(domainName);
return `https://favicon.twenty.com/${sanitizedDomain}`; return sanitizedDomain
? `https://favicon.twenty.com/${sanitizedDomain}`
: undefined;
}; };