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

[F] Implement Investigation Questions and Answers #11

Merged
merged 12 commits into from
Sep 13, 2023
Merged

Conversation

kylepratuch
Copy link
Contributor

@kylepratuch kylepratuch commented Aug 16, 2023

Included in this PR:

  • Question Entries. These aren't quite feature complete yet. There are two functional question/answer formats included here (text, select) and a third that is stubbed but not yet built out (fill in the blank). The idea here is to circle back and continue adding formats once our approach for data persistence is established.
  • Answer Element type. A fairly simple record type for storing users' responses to questions. There's no UI for these in Craft, but one can be added down the road, if desired.
  • Answer Element GQL support. Includes...
    A query: answers($userId, $investigationId)
    Two mutations: createAnswer($userId, $questionId, $investigationId, $data) and saveAnswer($id, $data)

TODO:

  • Implement GQL schema scoping configuration
  • Build out additional question types

@kylepratuch
Copy link
Contributor Author

@blnkt @dananjohnson I said I would get these changes merged to develop before heading out of town, but after rebasing this branch on top of develop and dealing with all of the conflicts in project config that are typical during a rebase, I'm not able to apply configuration changes in my local craft instance.

I'm reluctant to go forward with the merge while this is happening, so I pushed the rebase to a temporary branch: temp/rebase-kp-questions. I'll plan to finish sorting this out when I'm back, but the branch is there if y'all want or need to do this before I'm back.

@kylepratuch
Copy link
Contributor Author

Update: after restarting my container, all is well! Proceeding with the merge.

@kylepratuch kylepratuch merged commit df51702 into develop Sep 13, 2023
3 checks passed
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