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

feat(form): multiple files in file questions #1813

Closed
wants to merge 1 commit into from
Closed

Conversation

anehx
Copy link
Contributor

@anehx anehx commented Jul 6, 2022

Change the model to allow multiple files per answer to be stored. Migrate
the existing questions to the new question type, and migrate the existing
answers to the new schema.

Updated the GraphQL schema to allow saving and updating multiple files
on an answer. The answer's value is a list of dicts, each containing
a file name, and optionally an ID. When you're creating new files,
add an entry without ID to the value. Caluma will then save the file,
and return it back WITH and ID.

BREAKING CHANGE: This renames the question type constant for file questions,
and changes the semantics of the answer value for file questions as well:
It is now a list of dicts instead of a single string. The response type
for querying file(s) answers now also is a list instead of a single dict.

Change the model to allow multiple files per answer to be stored. Migrate
the existing questions to the new question type, and migrate the existing
answers to the new schema.

Updated the GraphQL schema to allow saving and updating multiple files
on an answer. The answer's value is a list of dicts, each containing
a file name, and optionally an ID. When you're creating new files,
add an entry without ID to the value. Caluma will then save the file,
and return it back WITH and ID.

BREAKING CHANGE: This renames the question type constant for file questions,
and changes the semantics of the answer value for file questions as well:
It is now a list of dicts instead of a single string. The response type
for querying file(s) answers now also is a list instead of a single dict.
@anehx
Copy link
Contributor Author

anehx commented Jul 6, 2022

Please merge this simultaneously with projectcaluma/ember-caluma#2040

@derrabauke
Copy link
Contributor

There is a question by @StephanH90 in the RFC to this feature. #1780 (comment)

Should we add the requested feature, @winged ?

@winged
Copy link
Contributor

winged commented Jul 13, 2022

This PR is missing some commits from the original PR, thus failing CI. I've re-created it here: #1821

So now we can close this one

@winged winged closed this Jul 13, 2022
@anehx anehx deleted the feat-multiple-files branch August 26, 2022 07:47
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.

3 participants