Fix packaging, timezone handling and HTML cleaning bugs #11

Merged

mathiaspalmqvist merged 1 commits from fix/bugs-and-packaging into main 2026-11-11 19:43:02 +00:00

Conversation 0 Commits 1 Files Changed 9

+207 -28

mathiaspalmqvist commented 2026-11-11 19:42:18 +00:00

Owner

Copy link

Copy source

• Replace symfony 7.4.x-dev constraints with stable ranges so the
  package installs with minimum-stability stable
• Add the Cision facade referenced in composer.json and a config file
  (slug, timezone); the constructor now falls back to config('cision.slug')
• Convert feed timestamps with a real timezone instead of a hardcoded +2h,
  which was off by an hour during winter time
• Return false from fetchContent() when the request fails or the page
  has no article/heading, instead of throwing
• Fix italic paragraph detection, which never matched
• Fix literal '\n' in HtmlCleaner whitespace cleanup
• Normalize empty feed elements to empty strings
• Update README and add tests for the above

- Replace symfony 7.4.x-dev constraints with stable ranges so the package installs with minimum-stability stable - Add the Cision facade referenced in composer.json and a config file (slug, timezone); the constructor now falls back to config('cision.slug') - Convert feed timestamps with a real timezone instead of a hardcoded +2h, which was off by an hour during winter time - Return false from fetchContent() when the request fails or the page has no article/heading, instead of throwing - Fix italic paragraph detection, which never matched - Fix literal '\n' in HtmlCleaner whitespace cleanup - Normalize empty feed elements to empty strings - Update README and add tests for the above

mathiaspalmqvist added 1 commit 2026-11-11 19:42:18 +00:00

Fix packaging, timezone handling and HTML cleaning bugs ... 392aec8529

- Replace symfony 7.4.x-dev constraints with stable ranges so the
  package installs with minimum-stability stable
- Add the Cision facade referenced in composer.json and a config file
  (slug, timezone); the constructor now falls back to config('cision.slug')
- Convert feed timestamps with a real timezone instead of a hardcoded +2h,
  which was off by an hour during winter time
- Return false from fetchContent() when the request fails or the page
  has no article/heading, instead of throwing
- Fix italic paragraph detection, which never matched
- Fix literal '\n' in HtmlCleaner whitespace cleanup
- Normalize empty feed elements to empty strings
- Update README and add tests for the above

mathiaspalmqvist merged commit 1141a4e382 into main 2026-11-11 19:43:02 +00:00

mathiaspalmqvist deleted branch fix/bugs-and-packaging 2026-11-11 19:43:05 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something isn't working

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

good first issue

Good for newcomers

help wanted

Extra attention is needed

invalid

This doesn't seem right

question

Further information is requested

wontfix

This will not be worked on

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

mathiaspalmqvist (Mathias Palmqvist)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: 559342-9607/laravel-cision-feed#11