Project

General

Profile

Actions
Copy link

Bug #2154

closed

Reporting broken in PT DNS

Added by Jonathan CLARKE almost 13 years ago. Updated over 12 years ago.

Status:
Released
Priority:
1 (highest)
Assignee:
Matthieu CERDA
Category:
Techniques
Target version:
2.3.6
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

R: @@ConfigureDNS@@result_repaired@@e2369d5e-a14a-4479-895f-76c831e27557@@258c793b-d87e-4cea-a05c-db91866da131@@1@@dnsConfiguration@@None@@2012-01-02 21:40:27+01:00##77b69cab-54e4-4054-b012-6445c2c438f6@#DNS settings were updated
R: @@ConfigureDNS@@result_success@@e2369d5e-a14a-4479-895f-76c831e27557@@258c793b-d87e-4cea-a05c-db91866da131@@1@@dnsConfiguration@@None@@2012-01-02 21:40:27+01:00##77b69cab-54e4-4054-b012-6445c2c438f6@#Every step completed successfully

As you can see from this log excerpt, the PT sent both a repaired and a success message...

Actions
Copy link
 #1

Updated by Jonathan CLARKE almost 13 years ago

  • Target version changed from 2.4.0~alpha3 to 2.3.6
Actions
Copy link
 #2

Updated by Nicolas CHARLES almost 13 years ago

  • Status changed from 2 to In progress
  • Assignee set to Nicolas CHARLES
Actions
Copy link
 #3

Updated by Nicolas CHARLES almost 13 years ago

  • Status changed from In progress to Pending technical review
  • % Done changed from 0 to 100

Applied in changeset commit:4545d987e8a47f9ff0e1bca4889970a4174f0738.

Actions
Copy link
 #4

Updated by François ARMAND almost 13 years ago

Applied in changeset commit:6b0417e55d27e8f4bd876d8982bd1efddde32781.

Actions
Copy link
 #5

Updated by Matthieu CERDA almost 13 years ago

Looks OK, might need a tiny cosmetic tweak.

Actions
Copy link
 #6

Updated by Jonathan CLARKE almost 13 years ago

Matthieu CERDA wrote:

Looks OK, might need a tiny cosmetic tweak.

Er, so... if it needs a tweak, please either explain what, or do it. Or should we just consider this OK?

Actions
Copy link
 #7

Updated by Nicolas CHARLES over 12 years ago

  • Assignee changed from Nicolas CHARLES to Matthieu CERDA

Matthieu, could you please explain what was the tweak you wished ?
Thank you

Actions
Copy link
 #8

Updated by Matthieu CERDA over 12 years ago

I commited this commit:59a0ddb2b28b7ac2ed862d55da1ddc94a7df1f06

It is just a small cosmetic change, I corrected the indentation to match the previous indentation pattern in the reports.

Actions
Copy link
 #9

Updated by Jonathan CLARKE over 12 years ago

  • Status changed from Pending technical review to Released

OK, this seems good now then.

Actions
Copy link

Also available in: Atom PDF