Project

General

Profile

Actions

Bug #2156

closed

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

Added by Jonathan CLARKE about 12 years ago. Updated almost 12 years ago.

Status:
Released
Priority:
3
Category:
Web - Config management
Target version:
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 #1

Updated by Jonathan CLARKE about 12 years ago

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

Updated by Jonathan CLARKE about 12 years ago

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

Actions #3

Updated by François ARMAND about 12 years ago

  • Assignee set to Arthur ANGLADE
Actions #4

Updated by François ARMAND about 12 years ago

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

Updated by Arthur ANGLADE about 12 years ago

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

Updated by Nicolas CHARLES about 12 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 #7

Updated by Jonathan CLARKE almost 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 #8

Updated by François ARMAND almost 12 years ago

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

Updated by François ARMAND almost 12 years ago

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

Updated by François ARMAND almost 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 #11

Updated by Jonathan CLARKE almost 12 years ago

  • Status changed from Pending technical review to Released
Actions

Also available in: Atom PDF