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.

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

2.8.1 #197

Merged
merged 19 commits into from
Nov 9, 2023
Merged

2.8.1 #197

merged 19 commits into from
Nov 9, 2023

Conversation

sarge1989
Copy link
Collaborator

PRs

#196
#194

Summary

  • Removed feedback options for interim message
  • Telegram handler moved to async as well, so it relies on same webhook handler as whatsapp and puts messages into queue
  • Batch jobs moved to GCP v2 functions runtime

bwsarge and others added 18 commits November 4, 2023 16:58
@bwsarge
put telegram handler into pubsub as well
90cb4a6
@bwsarge
updated npm run serve
049f0de
@bwsarge
removed feedback from interim responses
5bef4c5
@bwsarge
removed isInterimUseful field
3afd95e
@bwsarge
changed message
dedf1ff
@bwsarge
modified env.json
18e58c4
@bwsarge
updated userResponses
8fd81cd
@bwsarge
updated tests
df14443
@bwsarge
updated userresponses
57d7d89
@bwsarge
updated tests
76d5bb0
@bwsarge
Merge branch 'feature/telegram-pubsub' into feat/remove-interim-feedback
9d67fc9
@bwsarge
changed env variables
8febd1c
@bwsarge
updated integration tests for new referral prefix
090bbf8
@bwsarge
switched batch jobs to v2
fc8aca7
@bwsarge
Fixed mistake with session expiring batch job
34c785c
@bwsarge
changed copy
b4914cc
@sarge1989
Merge pull request #194 from CheckMateSG/feat/remove-interim-feedback
eff5c52 
Remove interim feedback and migrated Telegram handler to pubsub
@sarge1989
Merge pull request #196 from CheckMateSG/feat/batch-Jobs-V2-upgrade
175422e 
switched batch jobs to v2
@sarge1989
Copy link
Collaborator Author

Remember to delete v1 functions before proceeding

firebase functions:delete batchJobs

@sarge1989 sarge1989 merged commit dcc5866 into main Nov 9, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stonefruit stonefruit stonefruit approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sarge1989 @stonefruit @bwsarge