Skip to content
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.

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

Frontend checklist implementation #5460

Merged
merged 32 commits into from
Sep 12, 2024

Conversation

manuelmeister
Copy link
Member

@manuelmeister manuelmeister commented Jun 30, 2024

  • Admin ui
  • ContentNode Checklist

See example: https://bacluc-1.ecamp3.ch/
See feature toggle: https://pr5460.ecamp3.ch/camps/3c79b99ab424/GRGR/program/activities/12f34c89ce11

Frontend part of #4936

@manuelmeister manuelmeister added the deploy! Creates a feature branch deployment for this PR label Jun 30, 2024
Copy link

github-actions bot commented Jun 30, 2024

Feature branch deployment currently inactive.

If the PR is still open, you can add the deploy! label to this PR to trigger a feature branch deployment.

@manuelmeister manuelmeister added deploy! Creates a feature branch deployment for this PR and removed deploy! Creates a feature branch deployment for this PR labels Jun 30, 2024
@manuelmeister manuelmeister mentioned this pull request Jul 5, 2024
@manuelmeister manuelmeister changed the title Draft: Frontend checklist implementation (admin) Draft: Frontend checklist implementation Jul 5, 2024
@manuelmeister manuelmeister mentioned this pull request Jul 8, 2024
@manuelmeister manuelmeister added deploy! Creates a feature branch deployment for this PR and removed deploy! Creates a feature branch deployment for this PR labels Aug 27, 2024
@BacLuc
Copy link
Contributor

BacLuc commented Aug 27, 2024

works like a charm, thank you

image

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.

some minor adjustments

.helm/ecamp3/values.yaml Outdated Show resolved Hide resolved
.helm/ecamp3/values.yaml Show resolved Hide resolved
frontend/src/components/activity/content/Checklist.vue Outdated Show resolved Hide resolved
Copy link
Contributor

@simfeld simfeld left a comment

Choose a reason for hiding this comment

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

  • feature toggle works ✅
  • creating and editing checklists ✅
  • using checklists in schedule entries ✅
  • drag & drop causes backend errors ❌
An exception occurred while executing a query: SQLSTATE[23505]:
Unique violation: 7
ERROR: duplicate key value violates unique constraint "checklistitem_checklistid_parentid_position_unique" DETAIL: Key (checklistid, parentid, "position")=(c5163ea9d9a6, 9d13b06a9080, 2) already exists. 

@manuelmeister manuelmeister added this pull request to the merge queue Sep 12, 2024
@simfeld simfeld removed this pull request from the merge queue due to a manual request Sep 12, 2024
@simfeld
Copy link
Contributor

simfeld commented Sep 12, 2024

Now also displays checklists if more than 1 of them exist 👍 LGTM

@manuelmeister manuelmeister added this pull request to the merge queue Sep 12, 2024
Merged via the queue into ecamp:devel with commit faeb816 Sep 12, 2024
34 checks passed
@manuelmeister manuelmeister deleted the feature/checklist-frontend branch September 12, 2024 22:02
@manuelmeister manuelmeister mentioned this pull request Sep 12, 2024
@manuelmeister manuelmeister mentioned this pull request Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy! Creates a feature branch deployment for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants