Add the missing return after creating the new Report, make that line meaningless
Add the check whether container.name is an instance of an IdentifierStream node, otherwise, there will be an uncaught error when it's actually a PrimaryExpresssionNode.
Add the any assertion.
Issue
(issue link here)
Lasting Changes (Technical)
(please list down: code changes/things that have wide-effect; new libraries/functions added that can be used by others; examples below)
(Added class EmailValidator to validate email address' validity)
(Added Tenant#is_trial? check)
Checklist
Please check directly on the box once each of these are done
Summary
Fix the problems as pointed out in: https://holistics.slack.com/archives/C0278BDLPK8/p1706870126558909. NO breaking change is introduced!
container.nameis an instance of anIdentifierStreamnode, otherwise, there will be an uncaught error when it's actually aPrimaryExpresssionNode.anyassertion.Issue
(issue link here)
Lasting Changes (Technical)
(please list down: code changes/things that have wide-effect; new libraries/functions added that can be used by others; examples below)
class EmailValidatorto validate email address' validity)Tenant#is_trial?check)Checklist
Please check directly on the box once each of these are done