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

LRS-82 Allow empty contextActivities map #82

Merged

milt merged 1 commit into master from LRS-82

Jan 12, 2024

Member

milt commented Jan 12, 2024 •

edited by jira bot

Loading

LRS-82 Allow contextActivities to be an empty map.
previously the test was pointed at the array which was wrong


          LRS-82 Allow empty contextActivities map and fix test

f790daa

previously the test was pointed at the array which was wrong

milt requested review from kelvinqian00 and cliffcaseyyet

January 12, 2024 14:52

Collaborator

kelvinqian00 commented Jan 12, 2024

This was tested on SQL LRS and it works, and any other LRSs can be updated in tandem if they break, so this should be fine.

kelvinqian00 approved these changes

View reviewed changes

milt merged commit 135ad19 into master

2 checks passed

milt deleted the LRS-82 branch

January 12, 2024 15:24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet