Project

General

Profile

Actions
Copy link

Bug #2156

closed

Popup for list of policy instances in a CR contains PT's technical name

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

Status:
Released
Priority:
3
Assignee:
François ARMAND
Category:
Web - Config management
Target version:
2.3.7
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

In the CR screen, if a CR has multiple PIs, the table at the top displays the name of the first and some "...". When you click on this, you get a popup with a list of PIs, and the names of the PTs their based on.

These PT names are their technical names (ie dnsConfiguration) instead of their full human readable names.

Actions
Copy link
 #1

Updated by Jonathan CLARKE almost 13 years ago

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

Updated by Jonathan CLARKE almost 13 years ago

At the same time, it would be great to add the PT's version here.

Actions
Copy link
 #3

Updated by François ARMAND almost 13 years ago

  • Assignee set to Arthur ANGLADE
Actions
Copy link
 #4

Updated by François ARMAND almost 13 years ago

  • Target version changed from 2.4.0~alpha4 to 2.4.0~alpha5
Actions
Copy link
 #5

Updated by Arthur ANGLADE almost 13 years ago

  • Status changed from New to Pending technical review
  • % Done changed from 0 to 100
Actions
Copy link
 #6

Updated by Nicolas CHARLES almost 13 years ago

  • Status changed from Pending technical review to Released

That's correct, but could you please try to avoid commiting things like "test Close" Arthur ?
If you are not sure of what your commit will contain, you can type "git diff" to review your commit before push it. But it seems to have been corrected by another commit (or reverse ?)

Actions
Copy link
 #7

Updated by Jonathan CLARKE over 12 years ago

  • Status changed from Released to 2
  • Assignee changed from Arthur ANGLADE to François ARMAND
  • Target version changed from 2.4.0~alpha5 to 2.3.7
  • % Done changed from 100 to 90

This also affects branch 2.3, so can you backport it please François?

Actions
Copy link
 #8

Updated by François ARMAND over 12 years ago

  • Status changed from 2 to Pending technical review
Actions
Copy link
 #9

Updated by François ARMAND over 12 years ago

  • Status changed from Pending technical review to In progress
Actions
Copy link
 #10

Updated by François ARMAND over 12 years ago

  • Status changed from In progress to Pending technical review

To recap:

  • the first commit (2df153ea) seems to have been an error
  • the second (9035a564) is the actual correction, and was reviewed by NicoC
  • these two commits were done on branch 2.4
  • then, I backported the correction in branch 2.3 (73bd88cd)

So, it should be ok, and closable.

Actions
Copy link
 #11

Updated by Jonathan CLARKE over 12 years ago

  • Status changed from Pending technical review to Released
Actions
Copy link

Also available in: Atom PDF