Benature/omnivore
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #979 from omnivore-app/fix/logseq-weekly-newsletter

Browse Source 
fix not saving loqseq weekly newsletter due to no url
This commit is contained in:
Jackson Harper
2022-07-18 11:29:15 -07:00
committed by GitHub
parent badd32ed39 65f28fe708
commit 91d62ff9c9
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
packages/api/src/utils/parser.ts
View File

@ -444,7 +444,10 @@ export const isProbablyNewsletter = async (html: string): Promise<boolean> => {
}
// Check if this is a newsletter from revue
if (dom.querySelectorAll('img[src*="getrevue.co"]').length > 0) {
if (
dom.querySelectorAll('img[src*="getrevue.co"], img[src*="revue.email"]')
.length > 0
) {
const getrevueUrl = revueNewsletterHref(dom)
if (getrevueUrl) {
return true
@ -452,14 +455,11 @@ export const isProbablyNewsletter = async (html: string): Promise<boolean> => {
}
// Check if this is a convertkit.com newsletter
if (dom.querySelectorAll('img[src*="convertkit-mail.com"]').length > 0) {
const convertkitUrl = convertkitNewsletterHref(dom)
if (convertkitUrl) {
return true
}
}
return false
return (
dom.querySelectorAll(
'img[src*="convertkit.com"], img[src*="convertkit-mail.com"]'
).length > 0
)
}
const beehiivNewsletterHref = (dom: Document): string | undefined => {