Skip to content

fix: don't show url for non-public posts#77

Merged
ictbeheer merged 3 commits intomainfrom
feat/public-post-url
Feb 3, 2026
Merged

fix: don't show url for non-public posts#77
ictbeheer merged 3 commits intomainfrom
feat/public-post-url

Conversation

@ictbeheer
Copy link
Contributor

@ictbeheer ictbeheer commented Feb 2, 2026

This prevents urls like https://example.com?p=42

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Coverage report for commit: 8e77a6e
File: coverage.xml

Cover ┌─────────────────────────┐ Freq.
   0% │ ███████████████████████ │ 73.3%
  10% │ ███░░░░░░░░░░░░░░░░░░░░ │  6.7%
  20% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  30% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  40% │ ███░░░░░░░░░░░░░░░░░░░░ │  6.7%
  50% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  60% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  70% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  80% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  90% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
 100% │ █████░░░░░░░░░░░░░░░░░░ │ 13.3%
      └─────────────────────────┘
 *Legend:* █ = Current Distribution 
Summary - Lines: 11.54% | Methods: 7.84%
FilesLinesMethodsBranches
src/Attributes
   Meta.php--100.00%
   MetaPrefix.php--100.00%
   TaxonomyPrefix.php--100.00%
   Terms.php--100.00%
src/Contracts
   PostDataInterface.php100.00%100.00%100.00%
src/Enums
   PostStatus.php--100.00%
src
   ImageData.php--100.00%
   PostData.php9.82%7.69%100.00%
   TermData.php--100.00%
   UserData.php--100.00%
src/Mappers
   PostPrefixMapper.php--100.00%
   UserPrefixMapper.php--100.00%
src/Normalizers
   WPPostNormalizer.php--100.00%
src/Providers
   DataServiceProvider.php100.00%100.00%100.00%
src/Traits
   HasMeta.php36.00%-100.00%

🤖 comment via lucassabreu/comment-coverage-clover

@mvdhoek1
Copy link

mvdhoek1 commented Feb 3, 2026

Er faalt nog wel een phpstan test.

@ictbeheer ictbeheer merged commit 8da3e0b into main Feb 3, 2026
5 checks passed
@ictbeheer ictbeheer deleted the feat/public-post-url branch February 3, 2026 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants