-
Notifications
You must be signed in to change notification settings - Fork 274
[comp] Production Deploy #2359
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
Merged
[comp] Production Deploy #2359
Changes from all commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
cfa39e3
fix(integration-platform): filter GWS employees by org units and filt…
chasprowebdev 0408858
fix(integration-platform): remove duplcated user filtering logic acro…
chasprowebdev 3266427
Merge pull request #2358 from trycompai/chas/google-workspace-users
tofikwest b460cab
feat(questionnaire): update upload and parse functionality to trigger…
tofikwest 70c8d7e
fix(questionnaire): correct maxDuration values for parsing tasks
tofikwest e8937d1
feat(questionnaire): add fileSize to questionnaire upload and parsing
tofikwest e149d4e
chore(api): update .gitignore to include local scripts directory
tofikwest b670339
Merge pull request #2360 from trycompai/tofik/fix-knowledge-base-ques…
tofikwest 414475d
fix(questionnaire): update upload status message for async parsing tr…
tofikwest e53894b
Merge pull request #2361 from trycompai/tofik/upload-file-update-status
tofikwest efe5640
Merge branch 'release' into main
tofikwest 4884007
fix(questionnaire): update model name in logging for question parsing
tofikwest 8fd4607
Merge pull request #2362 from trycompai/tofik/fix-log-model-name
tofikwest File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Source field lost in async task flow
Low Severity
The
sourcefield fromUploadAndParseDto(which can be'internal'or'external') is no longer propagated to theparse-questionnairetask payload. The old synchronous flow explicitly passedsource: dto.source || 'internal'topersistQuestionnaireResult, but the new async flow omits it entirely from thetasks.triggercall. The task'sdb.questionnaire.createalso doesn't setsource, relying on the Prisma@default("internal"). This silently drops thesourcevalue for any caller passing'external'.Additional Locations (1)
apps/api/src/trigger/questionnaire/parse-questionnaire.ts#L351-L374