Skip to content

Target the shop newsletter plugin path - #10

Merged
alanw707 merged 1 commit into
mainfrom
fix/newsletter-popup-shop-path
Oct 12, 2026
Merged

alanw707 merged 1 commit into
mainfrom
fix/newsletter-popup-shop-path

Conversation

@alanw707

Copy link
Copy Markdown
Owner

Summary

  • derive the newsletter plugin path under the shop WordPress install
  • abort on any read failure instead of falling back to another site
  • compute the checked-in artifact hash for safe idempotent retries
  • bound lftp-interpolated local and key paths

Evidence

  • final subagent review: PASS
  • bash tests/scripts/deploy-file.test.sh passes
  • workflow YAML parse and git diff --check pass

Prior production attempts stopped before any plugin write: the first failed path validation and the second received 550 during the initial backup read.

@alanw707
alanw707 merged commit 785ecb0 into main Oct 12, 2026
7 checks passed
@alanw707
alanw707 deleted the fix/newsletter-popup-shop-path branch October 12, 2026 21:22
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.

1 participant