search

sdefresne / gerrit-monitor

Source for Gerrit Monitor Chrome extension
Apache License 2.0
21 stars 16 forks source link

Fix popup when a CL doesn't have revisions #53

Open avm99963 opened 8 months ago

avm99963 commented 8 months ago

When a CL doesn't have revisions, the current revision can't be found and thus we can't retrieve the current revision's description.

This CL fixes this by returning null in getDescription() when this happens, and falling back to the change's subject in the CL widget.

Fixes #52

avm99963 commented 8 months ago

@sdefresne Friendly ping in case you didn't see it (in another PR you mention you didn't receive an email notification for the PR, maybe tagging you sends it :)

Thanks in advance! :))