The default bug view has changed. See this FAQ.

Drop jquery*.min.js from test pilot

RESOLVED FIXED in Thunderbird 13.0

Status

Thunderbird
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Trunk
Thunderbird 13.0

Thunderbird Tracking Flags

(thunderbird12 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 598937 [details] [diff] [review]
The fix

Doing a few searches on the code for Test Pilot shows that we don't actually need the *.min.js files. These actually come from the shipped version of the "flot" extension to jquery which includes both the min and non-min files, so we should really just package the ones we need.
Attachment #598937 - Flags: review?(jdicarlo)
(Assignee)

Updated

5 years ago
Blocks: 729127
(Assignee)

Updated

5 years ago
Attachment #598937 - Flags: review?(jdicarlo) → review?(squibblyflabbetydoo)
(Assignee)

Updated

5 years ago
Attachment #598937 - Flags: review?(irving)
Comment on attachment 598937 [details] [diff] [review]
The fix

Review of attachment 598937 [details] [diff] [review]:
-----------------------------------------------------------------

Can't see any reason why we need these files.
Attachment #598937 - Flags: review?(irving) → review+
(Assignee)

Comment 2

5 years ago
Checked in: http://hg.mozilla.org/comm-central/rev/c04a8a9ed0f3
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0

Comment 3

5 years ago
Comment on attachment 598937 [details] [diff] [review]
The fix

Clearing out my review, since this is already checked in. Sorry about dropping the ball on this...
Attachment #598937 - Flags: review?(squibblyflabbetydoo)
(Assignee)

Comment 4

5 years ago
Comment on attachment 598937 [details] [diff] [review]
The fix

[Triage Comment]
Taking to beta as I want to get the version of test pilot we're distributing, synced up with the version on AMO.
Attachment #598937 - Flags: approval-comm-aurora+
(Assignee)

Updated

5 years ago
Attachment #598937 - Flags: approval-comm-aurora+ → approval-comm-beta+
(Assignee)

Comment 5

5 years ago
Checked in:

http://hg.mozilla.org/releases/comm-beta/rev/7196a9fd07f4
status-thunderbird12: --- → fixed
You need to log in before you can comment on or make changes to this bug.