show diff against base table when sending e-mail alerts for scheduled changes

NEW
Unassigned

Status

P3
normal
a year ago
a year ago

People

(Reporter: bhearsum, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [lang=python][ready])

(Reporter)

Description

a year ago
Currently we send mail such as:
Row to be inserted:
{'base_alias': 'firefox-release',
 'base_backgroundRate': 100,
 'base_buildID': None,
 'base_buildTarget': None,
 'base_channel': 'release',
 'base_comment': 'default release rule updated by buildbot, DO NOT DELETE',
 'base_data_version': 147,
 'base_distVersion': None,
 'base_distribution': None,
 'base_fallbackMapping': None,
 'base_headerArchitecture': None,
 'base_locale': None,
 'base_mapping': 'Firefox-54.0-build3-whatsnew',
 'base_osVersion': None,
 'base_priority': 90,
 'base_product': 'Firefox',
 'base_rule_id': 145,
 'base_systemCapabilities': None,
 'base_update_type': 'minor',
 'base_version': None,
 'change_type': 'update',
 'csrf_token': '1498007702##fe6927e9a30927fb3b0baebef3ea7f86bf04e314',
 'data_version': 1,
 'scheduled_by': 'jwood@mozilla.com'}

Unfortunately, scheduled changes are not terribly useful without context. The most useful thing to know is "what is this scheduled change going to change". In the above case, backgroundRate and fallbackMapping were different vs. the base, so that's what should've been highlighted.
I also brought up in IRC that I'd like the ability to see "what would change" in the UI too, not just the whole-new-rule.  (Basically the ability to do the diff that e-mail sends as a UI thing too)
(Reporter)

Comment 2

a year ago
(In reply to Justin Wood (:Callek) from comment #1)
> I also brought up in IRC that I'd like the ability to see "what would
> change" in the UI too, not just the whole-new-rule.  (Basically the ability
> to do the diff that e-mail sends as a UI thing too)

Agreed. This is https://bugzilla.mozilla.org/show_bug.cgi?id=1301725.
(Reporter)

Updated

a year ago
Mentor: bhearsum
Priority: P2 → P3
Whiteboard: [lang=python][ready]
(Reporter)

Updated

a year ago
Mentor: bhearsum
You need to log in before you can comment on or make changes to this bug.