prehistoric-systems/omnivore
1
0
Fork 0
mirror of https://github.com/omnivore-app/omnivore.git synced 2026-03-11 08:54:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #1638 from omnivore-app/fix/no-author-in-newsletter

Browse source 
Fix the error when no name in subscription by using the email address as the author when no author found in the newsletter
This commit is contained in:
Hongbo Wu 2023-01-10 12:49:53 +08:00 committed by GitHub
commit 4caeab251d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/content-handler/src/content-handler.ts
View file

@ -134,7 +134,7 @@ export abstract class ContentHandler {
// e.g. 'Jackson Harper from Omnivore App <jacksonh@substack.com>'
// or 'Mike Allen <mike@axios.com>'
const parsed = addressparser(from)
if (parsed.length > 0) {
if (parsed.length > 0 && parsed[0].name) {
return parsed[0].name
}
return from

5
packages/content-handler/test/newsletter.test.ts
View file

@ -93,6 +93,11 @@ describe('Newsletter email test', () => {
const from = 'Mike Allen <mike@axios.com>'
expect(new AxiosHandler().parseAuthor(from)).to.equal('Mike Allen')
})
it('returns email address if author is not there', () => {
const from = 'mike@axios.com'
expect(new AxiosHandler().parseAuthor(from)).to.equal(from)
})
})
describe('getNewsletterHandler', () => {