Skip to content

issues/1010-1013: security headers, devcontainer, husky hooks, dev po… - #1043

Merged
Smartdevs17 merged 1 commit into
Smartdevs17:mainfrom
Daniel71250:issues/1010-1013-security-headers-devcontainer-hooks-developer-portal
Sep 1, 2026
Merged

issues/1010-1013: security headers, devcontainer, husky hooks, dev po…#1043
Smartdevs17 merged 1 commit into
Smartdevs17:mainfrom
Daniel71250:issues/1010-1013-security-headers-devcontainer-hooks-developer-portal

Conversation

@Daniel71250

Copy link
Copy Markdown

…rtal tests

Pull Request Checklist

Quality Gates (All must pass before merge)

  • Lint: Code passes ESLint and Prettier checks
  • Type Check: TypeScript compilation succeeds
  • Tests: All tests pass
  • Build: Project builds successfully
  • Rust Format: Smart contract formatting is correct
  • Rust Clippy: Smart contract linting passes
  • Rust Tests: All smart contract tests pass
  • Rust Build: Smart contracts compile successfully

Additional Requirements

  • New code has appropriate TypeScript types
  • No hardcoded secrets or credentials
  • New features have corresponding tests
  • Documentation updated if needed

Reviewers

  • At least 1 approval required for merge
  • All CI checks must be green

This PR will not be mergeable until all quality gates pass.
closes
closes #1010
closes #1011
closes #1012
closes #1013

…rtal tests

- Smartdevs17#1010 add HSTS + Permissions-Policy security headers in backend and app.json
- Smartdevs17#1013 add .devcontainer/devcontainer.json and .gitpod.yml
- Smartdevs17#1012 wire commitlint into .husky/commit-msg (lint-staged + husky already present)
- Smartdevs17#1011 add unit tests for developer portal service and store
- clean .gitignore for snapshots and test artifacts
@drips-wave

drips-wave Bot commented Aug 27, 2026

Copy link
Copy Markdown

@Daniel71250 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Smartdevs17
Smartdevs17 merged commit 5d12995 into Smartdevs17:main Sep 1, 2026
1 check passed
@Smartdevs17

Copy link
Copy Markdown
Owner

Thanks for contributing! The changes have been merged. Feel free to leave a review or feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants