Closed Bug 1083768 Opened 6 years ago Closed 6 years ago

IRC auth messages showing up on moznet

Categories

(Chat Core :: IRC, defect)

Product:
Chat Core
Component:
IRC
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: clokep, Assigned: clokep)

References

Details

Attachments

(2 files)

Patch v1
1.43 KB, patch
florian
: review+
Details | Diff | Splinter Review
Avoid error
1.34 KB, patch
nhnt11
: review+
Details | Diff | Splinter Review 
https://hg.mozilla.org/comm-central/rev/2505dd518163#l2.1 made a change to the way we display NOTICE messages to the user (in particular for AUTH).
Attached patch Patch v1 — — Splinter Review
Attachment #8506141 - Flags: review?(aleth)
Comment on attachment 8506141 [details] [diff] [review]
Patch v1

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

This seems reasonable and I'm pretty sure you know what you are doing :-).
Attachment #8506141 - Flags: review?(aleth) → review+
Attached patch Avoid error — — Splinter Review
Attachment #8506182 - Flags: review?(nhnt11)
Comment on attachment 8506182 [details] [diff] [review]
Avoid error

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

Thanks!
Attachment #8506182 - Flags: review?(nhnt11) → review+
https://hg.mozilla.org/comm-central/rev/df2b37118b36
https://hg.mozilla.org/comm-central/rev/10f2002a35d8
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.6
Blocks: 1184406
You need to log in before you can comment on or make changes to this bug.