-
Notifications
You must be signed in to change notification settings - Fork 4
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: AI-categorized docs show in strategy sorts (PT-188476611) #2433
feat: AI-categorized docs show in strategy sorts (PT-188476611) #2433
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2433 +/- ##
=======================================
Coverage 86.36% 86.37%
=======================================
Files 746 746
Lines 38659 38659
Branches 9868 9868
=======================================
+ Hits 33388 33390 +2
+ Misses 4972 4970 -2
Partials 299 299
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
collaborative-learning Run #14137
Run Properties:
|
Project |
collaborative-learning
|
Branch Review |
188476611-ai-categorized-docs-show-in-strategy-sorts
|
Run status |
Passed #14137
|
Run duration | 14m 48s |
Commit |
b67f948ba8: chore: reinstate existing tag-handling code
|
Committer | Ethan McElroy |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
3
|
Skipped |
0
|
Passing |
117
|
View all changes introduced in this branch ↗︎ |
We'll remove this in a separate PR 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.
Looks good to me. 👍
#188476611
Adds a Cloud Function that will watch for comments and update the related document's strategies array with tag values in the comment. The function will also delete values from the document's strategies array if a comment containing tags is deleted and no other comments contain the same tags.
Also included is a small change to the
emulator
script infunctions-v2
. It will now save and load data from the newfunctions-v2/emulator-data
directory so that state will persist between emulator runs.