Skip to content

Update Permission#26

Merged
devCluna merged 16 commits intoproductionfrom
main
Apr 13, 2026
Merged

Update Permission#26
devCluna merged 16 commits intoproductionfrom
main

Conversation

@devCluna
Copy link
Copy Markdown
Owner

No description provided.

devCluna added 16 commits April 3, 2026 21:05
Runs on every push and PR across all branches. Checks:
- TypeScript (server + admin)
- Build
- Strapi plugin verify
- Version not already published on npm
- Dry run publish

publish.yml now requires CI to pass before publishing to npm.
- Fix test:ts:front and test:ts:back scripts to use npx tsc (was yarn-only syntax)
- Add explicit types for implicit any parameters caught by the type check:
  MouseEvent on DropZone drag handle click
  boolean on Modal.Root onOpenChange in EmbedModal and FormPreview
  string | number on SingleSelect onChange in FieldSettingsPanel and SubmissionsPage
…gnore

Rollup was resolving register.js (and other compiled .js files) instead
of the .ts sources, causing 'default is not exported' build failures in CI.
Deleted all .js artifacts from server/src and admin/src and added them
to .gitignore so they are never committed again.
…tion

Without --noEmit, tsc outputs .js files next to the .ts sources.
Rollup then resolves the CJS .js files instead of the .ts sources and
fails with 'default is not exported' on the CommonJS interop boundary.
Push trigger now only fires on main, development and production.
Feature branches are covered by the pull_request trigger alone,
so CI no longer runs twice when pushing to an open PR.
- Version 1.0.0 (initial public release)
- Add description, keywords, repository, bugs and homepage fields
- Set strapi.displayName to 'Form Builder CMS'
Calling ci.yml as a reusable workflow while it also triggers independently
on push to production caused a workflow file error (0s failure).
Inlined all CI steps directly into publish.yml to avoid the conflict.
@devCluna devCluna merged commit 5c1d039 into production Apr 13, 2026
1 of 3 checks passed
@devCluna devCluna deleted the main branch April 13, 2026 15:32
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 13, 2026

Caution

Review failed

The head commit changed during the review from b382e83 to d447d6c.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch main

Comment @coderabbitai help to get the list of available commands and usage tips.

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