Address dynamic groups PR comments

[sashankaryal]

What changes are proposed in this pull request?

Address comments from #2934

How is this patch tested? If it is not, please explain why.

Locally

Release Notes

Is this a user-facing change that should be mentioned in the release notes?

• No. You can skip the rest of this section.
• Yes. Give a description of this change to be included in the release
  notes for FiftyOne users.

What areas of FiftyOne does this PR affect?

• App: FiftyOne application changes
• Build: Build and test infrastructure changes
• Core: Core fiftyone Python library changes
• Documentation: FiftyOne documentation changes
• Other

[ritch]

Consider:

({ subBar = null })
// ...
{subBar}

[codecov]

Codecov Report

Patch coverage: 58.33% and no project coverage change.

Comparison is base (2a0b837) 62.21% compared to head (9029a40) 62.21%.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3000   +/-   ##
========================================
  Coverage    62.21%   62.21%           
========================================
  Files          250      250           
  Lines        45536    45543    +7     
  Branches       319      319           
========================================
+ Hits         28328    28335    +7     
  Misses       17208    17208           

Flag	Coverage Δ
app	48.57% <58.33%> (+0.01%)	⬆️
python	99.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
app/packages/state/src/recoil/groups.ts	48.59% <58.33%> (+0.85%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.