Skip to content

Conversation

@jasonbahl
Copy link
Contributor

@jasonbahl jasonbahl commented Nov 28, 2023

What does this implement/fix? Explain your changes.

This fixes a bug where ACF Options Pages that are registered with a custom post_id were not resolving.

Does this close any currently open issues?

closes #128

Any relevant logs, error output, GraphiQL screenshots, etc?

Before:

CleanShot 2023-11-28 at 08 37 01

After:

CleanShot 2023-11-28 at 08 32 44

@jasonbahl jasonbahl self-assigned this Nov 28, 2023
@jasonbahl jasonbahl mentioned this pull request Nov 28, 2023
3 tasks
@coveralls
Copy link

Pull Request Test Coverage Report for Build 502deaa9b0f5788d037f3d05b36e423a517930c8-PR-132

  • 4 of 6 (66.67%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.01%) to 62.555%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/Utils.php 2 4 50.0%
Totals Coverage Status
Change from base Build 8ae6cd020248cbcd8d832eabd9a75472a0716adf: 0.01%
Covered Lines: 1993
Relevant Lines: 3186

💛 - Coveralls

@jasonbahl jasonbahl merged commit b56bef5 into main Nov 28, 2023
@jasonbahl jasonbahl deleted the fix/128-options-page-with-custom-post_id-not-resolving branch January 24, 2024 20:02
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.

Options Fields Return Null

3 participants