Skip to content

Conversation

@AliKdhim87
Copy link
Contributor

No description provided.

@AliKdhim87 AliKdhim87 requested a review from Robbert August 21, 2025 14:41
@vercel
Copy link

vercel bot commented Aug 21, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
cms-frameless-io Error Error Aug 22, 2025 2:26pm
tiptap Ready Ready Preview Comment Aug 22, 2025 2:26pm

@codecov
Copy link

codecov bot commented Aug 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@Robbert Robbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

De documentatie staat er nu 2 keer in: ik zou het alleen in strapi.frameless.io zetten, en vanuit de README in de root daar naar verwijzen

Comment on lines 50 to 74
### Local Environment Testing

#### Non-Docker Testing

Test development and build processes without Docker:

```bash
# Interactive development setup
yarn development
# Select: Dev → Choose app/project → Test functionality

# Interactive build setup
yarn development
# Select: Build → Choose app/project → Verify build success
```

#### Docker Environment Testing

Test both development and production Docker environments:

```bash
# Interactive Docker setup
yarn development
# Select: Docker → Choose project → Choose environment (dev/prod) → Choose action (up --build)
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dit stuk zou ik eruit laten; het is nogal obvious, en dan heb je minder risico dat deze documentatie out of sync gaat lopen

- Users & Permissions
- Settings panels
- strapi-plugin-config-sync
- All other installed plugins (ensure comprehensive testing)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kunnen we dit lijstje samenvoegen met functionality testing? Lijkt me dat er veel overlap zit. Bijvoorbeeld:

- [ ] Media upload and management with the Media Library

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

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

3 participants