Skip to content

Feature/activity load selected checklist items as collection #7199

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: devel
Choose a base branch
from
Open

Feature/activity load selected checklist items as collection #7199

wants to merge 3 commits into from

Conversation

pmattmann
Copy link
Member

ContentNode/ChecklistNodes: ChecklistItems-Collection-Link
Collection-Link allows to load all selected ChecklistItems in one Request

@pmattmann pmattmann requested a review from a team April 12, 2025 10:01
@pmattmann pmattmann requested a review from a team April 12, 2025 10:09
@simfeld simfeld added the deploy! Creates a feature branch deployment for this PR label Apr 12, 2025
@simfeld simfeld temporarily deployed to feature-branch April 12, 2025 10:32 — with GitHub Actions Inactive
@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 12, 2025 •
edited
Loading

✅ Feature branch deployment ready!

Name Link
😎 Deployment https://pr7199.ecamp3.ch/
🔑 Login test@example.com / test
🕒 Last deployed at Sat Apr 12 2025 12:33:36 GMT+0200
🔨 Latest commit 6c9cdbbdbebc861c82d2b136bbade1765a3d1df8
🔍 Latest deploy log https://github.com/ecamp/ecamp3/actions/runs/14418754492/job/40438756764
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

BacLuc
BacLuc requested changes Apr 12, 2025
View reviewed changes
Copy link
Contributor

@BacLuc BacLuc left a comment

Choose a reason for hiding this comment

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

there is silly behavior with the checklist overview

@BacLuc
Copy link
Contributor

BacLuc commented Apr 13, 2025 •
edited
Loading

Maybe part of your problem is solved with 82ac728
in

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

@BacLuc BacLuc BacLuc requested changes

@usu usu usu approved these changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
deploy! Creates a feature branch deployment for this PR Performance
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pmattmann @BacLuc @usu @simfeld