Skip to content

fix: make production app deployable - #2

Merged
JhumanJ merged 1 commit into
mainfrom
codex/fix-production-app
Jul 28, 2026
Merged

fix: make production app deployable#2
JhumanJ merged 1 commit into
mainfrom
codex/fix-production-app

Conversation

@JhumanJ

@JhumanJ JhumanJ commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

What changed

  • replace the invalid app metadata file with a valid Make Base configuration
  • inherit self-hosted base URLs, authorization, error handling, and token sanitization
  • fix Laravel form pagination and attached-webhook detach mapping
  • target the real production Make app ID
  • expose optional editable-submission links and optimize the logo below Make's 500 kB limit
  • add contract tests and GitHub Actions coverage

Root cause

The repository mixed app metadata with Make's Base code, referenced response paths that do not match Make's attached-webhook contract or Laravel's paginated resource envelope, and still targeted the placeholder production app ID.

Impact

The app can now be deployed from the repository, list forms beyond the first page, cleanly unregister webhooks, support self-hosted OpnForm instances, and keep API keys out of logs.

Validation

  • npm test — 6 contract tests passed
  • all 13 Make component files validated with Make Apps Editor 2.8.0 schemas
  • git diff --check
  • logo verified at 512×512 and 214,765 bytes

@JhumanJ
JhumanJ marked this pull request as ready for review July 28, 2026 15:01
@JhumanJ
JhumanJ merged commit 6b0af9c into main Jul 28, 2026
1 check passed
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