https://fedorahosted.org/freeipa/ticket/1477
Redirection after updating empty DNS Record (which is deleted).
Added hook to details facet for post update operation.
Petr
>From aafbaf2464dcc16f552290f72424170c42055b45 Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Fri, 22 Jul 2011 13:24
or handler.
Is it ACKable, or is it better to do it as described?
Petr
>From 3601c857fd4425c3df998e66a39235b67c441813 Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Tue, 26 Jul 2011 09:59:19 +0200
Subject: [PATCH] Fixed adding host without DNS reverse zone
https://fedorahosted.org/f
On Tue, 2011-07-26 at 21:32 -0400, Adam Young wrote:
> On 07/26/2011 07:09 PM, Endi Sukma Dewata wrote:
> > On 7/26/2011 6:27 AM, Petr Vobornik wrote:
> >> Fixed adding host without DNS reverse zone
> >>
> >> https://fedorahosted.org/freeipa/ticket/1481
>
it still works as intended.
>
>
> Missed a few files in my commit.
> ___
> Freeipa-devel mailing list
> Freeipa-devel@redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel
>From 75cc2819fafefc19d3feec7daf63b5bbe0
On Mon, 2011-08-01 at 23:03 -0400, Adam Young wrote:
> On 08/01/2011 10:26 PM, Adam Young wrote:
> > On 08/01/2011 03:19 PM, Rob Crittenden wrote:
> > > Ade Lee from the dogtag team looked at our installer and found
> > > that we restarted the pki-cad process too many times. Re-arranging
> > > so
new version attached
On Fri, 2011-07-29 at 12:11 -0500, Endi Sukma Dewata wrote:
> On 7/29/2011 11:12 AM, Petr Vobornik wrote:
> > There was a small error in add.js:162. Fixed!
>
> Nice job on the dialog boxes.
>
> There's a problem though, the Retry doesn
On Thu, 2011-08-04 at 13:24 -0500, Endi Sukma Dewata wrote:
> On 8/4/2011 4:22 AM, Petr Vobornik wrote:
> > new version attached
>
> Almost there, just a few more minor issues.
>
> >> Also these changes should be reverted back to maintain the Ajax context:
> >
ifications in error handling are OK.
If it really works in IE I would consider it ACKed.
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
7; doesn't exist any more so getting fqdn from field in
host.js:280 "fqdn: that.get_field('fqdn').save()" isn't functional.
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
On 08/09/2011 08:04 PM, Endi Sukma Dewata wrote:
On 8/9/2011 7:12 AM, Petr Vobornik wrote:
NACK
"Add and edit" without existing DNS reverse zone don't redirect to
edit page.
Field 'fqdn' doesn't exist any more so getting fqdn from field in
host.js:280 &quo
@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
Same repetitive code as in host adder dialog (1457). But it should be
covered by https://fedorahosted.org/freeipa/ticket/1394.
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa
@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
I don't like that there is no pattern in image file naming. Sometimes it
uses dashes, sometimes underscores. Some file names are capitalized. But
this isn't the subject of this patch (ticket).
--
Pet
t but checking update dns' (1592)
- added support(element's classes) for css styling of aggregated errors
- except search dialog delete (1592) - no other batch command uses this
feature (has to be explicitly turned on).
--
Petr Vobornik
From 52254b91e3c2546fea01c18f8bc879316ae7bb3d Mon Sep 1
devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
uld be moved into IPA.error_dialog so the
IPA.batch_command doesn't need to hold the 'errors' list.
left in batch_command for possible future use in custom handlers
6. The list of errors should be displayed as a list (with bullets) like
in the deleter dialog.
done
7. Just
patch from code and working perspective is OK.
Has this patch assigned some ticket? Don't know exactly what is IPA's
policy for submitting patches without tickets in Trac (for future tracking).
--
Petr Vobornik
___
Freeipa-devel mailing li
@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
data in success handler should not be undefined.
--
Petr Vobornik
From 7670faa42492e8850cffd7c54d60f6f6eaa7c51b Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Wed, 17 Aug 2011 15:06:41 +0200
Subject: [PATCH] Show error in serial association
https://fedorahosted.org/freeipa/ticket/1628
---
in
On 08/17/2011 05:38 PM, Petr Vobornik wrote:
Ticket #1628 - https://fedorahosted.org/freeipa/ticket/1628
Unreported insufficient access error
This patch is dependant on
freeipa-pvoborni-0004-1-error-dialog-for-batch-command.patch.
This may be only a checking if approach of this patch is good
ows in table. But
I sense, it could be useful later.
As I was implementing it, I noticed that after removing or adding sudo
option it refreshes facet -> all other associations. Don't know if it's
a feature or a bug. (sudo.js: 679, 683, 723, 727).
--
Petr Vobornik
From b2113a79520dce7e9
On 08/18/2011 10:28 AM, Petr Vobornik wrote:
On 08/17/2011 05:38 PM, Petr Vobornik wrote:
Ticket #1628 - https://fedorahosted.org/freeipa/ticket/1628
Unreported insufficient access error
This patch is dependant on
freeipa-pvoborni-0004-1-error-dialog-for-batch-command.patch.
This may be only
eipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK if leaving string in internal.py is the right thing to do when
deleting related label (the string isn't used anywhere else).
--
Petr Vobornik
On 08/18/2011 10:30 PM, Endi Sukma Dewata wrote:
On 8/18/2011 8:18 AM, Petr Vobornik wrote:
https://fedorahosted.org/freeipa/ticket/1639
- added unchecking all checkboxes of table_widget and its derivates to
on_load method.
line '$('input[name=select]', that.tbody).attr(&
On 08/18/2011 06:25 PM, Endi Sukma Dewata wrote:
On 8/17/2011 10:38 AM, Petr Vobornik wrote:
Ticket #1628 - https://fedorahosted.org/freeipa/ticket/1628
Unreported insufficient access error
This patch is dependant on
freeipa-pvoborni-0004-1-error-dialog-for-batch-command.patch.
This may be
On 08/22/2011 07:05 PM, Endi Sukma Dewata wrote:
On 8/22/2011 10:06 AM, Petr Vobornik wrote:
'Failed' moved to command. On 'failed' success is transformed to error -
can be change behaviour of serial associator in some commands
(previously some commands were executed ev
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com
ries. Thus, the IPA.rule_association_table_widget is
no longer needed.
Ticket #1692
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
--
Petr Vobornik
On 08/23/2011 11:09 PM, Endi Sukma Dewata wrote:
On 8/23/2011 6:34 AM, Petr Vobornik wrote:
Please take a look at the attached patch. This should be applied on top
of your patch. It does several things:
1. As mentioned over IRC, we should be treating these partial failure as
a success (the
x27;t executed.
Fixed integer minimum and maximum value checking.
Read-only and non-writeable fields are no longer considered required.
--
Petr Vobornik
From 8ccfd1ac82eaa2cb322a23166be54836f907c644 Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Wed, 24 Aug 2011 15:36:48 +0200
Subject: [PATCH] Valid
/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
there's a failure the rest of the commands will not be executed.
* This can be fixed by putting the commands into a batch and execute
them at once.
--
Petr Vobornik
From cfa78cfbef6130739f4dd161a33d4f535bd0fc02 Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Thu, 25 Aug 2011 14:57:44
ipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
- didn't thoroughly check ipa_init.json. I assume it's valid because
it's generated.
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat
@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
https://fedorahosted.org/freeipa/ticket/1689
Currently adding or deleting sudo options will refresh the entire page.
It's not a problem but the code could be optimized to refresh only the
sudo options table
--
Petr Vobornik
From 05bfd5e1fa5126ee581273d1adfcf995da4d4dcd Mon Sep 17 00:
On 08/26/2011 11:04 PM, Endi Sukma Dewata wrote:
On 8/26/2011 11:41 AM, Petr Vobornik wrote:
https://fedorahosted.org/freeipa/ticket/1689
Currently adding or deleting sudo options will refresh the entire page.
It's not a problem but the code could be optimized to refresh only the
sudo op
@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
facet.
'Reset' and 'Updates' buttons in details facet are enabled only if facet
is dirty.
Removes the problem above and 'no modification to be performed' annoyance.
--
Petr Vobornik
From 6b373a80c7c9741c36e1151ca9f89dcd79dd52dd Mon Sep 17 00:00:00 2001
From: Petr Voborni
translated message or a default message if not found.
Ticket #1701
for the changes: ACK
I found one more string: "'Error: '+error_thrown.name+'';"
entity.js:126
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@red
name - something similar to what is used now in
association_table) and do not blindly copy entity from facet as it is
done now (association entity often isn't equal to facet entity (eg in
memberof association)). This could be part of
https://fedorahosted.org/freeipa/ticket/1690 .
--
/mailman/listinfo/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
makes it more difficult to use.
--
Petr Vobornik
From d54e3d5758771fd010ce60ff0d77ee51964f7bc3 Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Wed, 31 Aug 2011 14:42:33 +0200
Subject: [PATCH] Attributes table not scrollable
https://fedorahosted.org/freeipa/ticket/1739
The attributes table
object is selected to be deleted or not, or even
if there is no object to be selected to delete.
One can click on this button...but then - there is no message indicating
that something should be selected for deletion for this button to do
anything.
Milestone: 3.0 Core Effort Backlog
--
Petr Vob
https://fedorahosted.org/freeipa/ticket/1767
Opening IPA Server/Entitlements causes: "Uncaught TypeError: Cannot call
method 'addClass' of undefined" error - Details.js:489
--
Petr Vobornik
From 17a3dbed887a75fa1ff0198edb5ae9873b1b35a3 Mon Sep 17 00:00:00 2001
From: Petr
by spec object usage.
It doesn't add any functionality.
3) IPA.permission_details_facet can be deleted - it isn't used anywhere.
Should we unite label align? In add dialog labels are aligned left, in
details table right.
Otherwise it looks OK.
--
Pet
On 09/08/2011 06:51 PM, Endi Sukma Dewata wrote:
On 9/8/2011 11:13 AM, Petr Vobornik wrote:
In IPA.details_table_section:
1)not renamed list_section_create method
Fixed.
Code clean-up in aci.js:
2) IPA.rights_section can be deleted and replaced by spec object usage.
It doesn't ad
indirectly via groups.
This does not apply to the command and run-as tables because they
do not support such option.
Ticket #1768
Wrong email title. It should be patch #266.
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
On 09/07/2011 09:06 PM, Endi Sukma Dewata wrote:
On 9/7/2011 7:16 AM, Petr Vobornik wrote:
https://fedorahosted.org/freeipa/ticket/1640
..
One issue, in HBAC/sudo rules details page if the category is changed
from 'all' to 'specific', the Delete button will be enabled
ds in details facet. As you write, some
clean-up should be addressed in #1394 or/and in #1696 (to be consistent
with dialogs).
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
t uses default label retrieval method in widget
(IPA.get_entity_param) instead of metadata.objects.[other_entity].label.
This isn't entirely wrong, but param labels aren't always consistent
with entity.labels.
Otherwise its OK.
--
Petr Vobornik
__
https://fedorahosted.org/freeipa/ticket/1788
Removed code duplication of undo links.
Simplified code of widget creation to be more readable.
--
Petr Vobornik
From c1e47469cd394c8934e0a6bf3cc84e88b5a6bb5a Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Tue, 13 Sep 2011 13:53:54 +0200
Subject
On 09/13/2011 08:04 AM, Endi Sukma Dewata wrote:
The labels from entity parameter are actually more appropriate. I've
updated the patch to use them instead. I also fixed some of the labels
(the run-as group label & doc is incorrect).
ACK
--
Petr
https://fedorahosted.org/freeipa/ticket/1787
In 'IPA Server/RBAC/Permission/Settings/Rights' is missing a option for
setting 'read' permission which is supported in CLI.
--
Petr Vobornik
From 6110e275e36adf310fc56d3d72480b1512a76be2 Mon Sep 17 00:00:00 2001
From: Petr Vob
Forgot to update tests - to address newly added validation row in
table_widget.
--
Petr Vobornik
From 40382df3620607760e8a6033b93b178d149f9ed4 Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Wed, 14 Sep 2011 13:01:25 +0200
Subject: [PATCH] Fixed: Some widgets do not have space for
x27;t be in the future (but there would have to be changes in save
method). So if its OK, after a test correction I would consider it ACKed.
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ight want to use this with non-boolean attributes. So
we need to be able to map a pair of non-boolean values into checked and
unchecked state during load()/reset() and vice versa during save().
--
Endi S. Dewata
ACK
--
Petr Vobornik
___
Freeipa-de
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
etween fqdn and other section
OK? I don't mind it, just wondering.
6) group.js:100 param_info contains invalid string "Create as a
non-POSIX group" for nonposix checkbox usage.
--
Petr Vobornik
___
Freeipa-devel mailing list
/listinfo/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
On 09/21/2011 10:10 PM, Endi Sukma Dewata wrote:
> On 9/21/2011 6:50 AM, Petr Vobornik wrote:
>
> Fixed. The dialog fields don't need undo, so the text() needs to be
> overridden to disable undo. This can be improved again later.
The override isn't necessary because it wasn&
be functional it would be IMO bad - checking line
doesn't imply that user wants it to be deleted right now. Even if
marking the line could be used only for deletion.
This code should be cleaned in the future (erased or fixed).
--
Petr Vobornik
__
1797
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat
On 09/22/2011 09:15 PM, Endi Sukma Dewata wrote:
I've split the patch into two: 4a is the revised original patch,
ACK
4b is the additional cleanup for the undo.
you missed: dns.js:562 and host.js:406 . Beside this it's OK.
--
Pet
_decode = function (value) {
if(value) {
return $('').html(value).text();
} else {
return '';
}
}
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
s
the filter automatically because that might not be the user's intention.
Ticket #1819
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
:
* element is added in order to set padding for cell text
because parent element has css-hacked width. Tbody's use of
overflow is the cause of used hacks.
--
Petr Vobornik
From 6d5429191f5f530d5aae6b689a2326404ef4f126 Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Fri, 23 Sep 2011 17:49:46
On 09/26/2011 08:15 AM, Endi Sukma Dewata wrote:
On 9/23/2011 11:05 AM, Petr Vobornik wrote:
https://fedorahosted.org/freeipa/ticket/1841
The column header for the attributes table (IPA.attributes_widget) does
not cover the entire width of the table. This problem appears in the
adder dialog
rules, this
attribute should be displayed as radio buttons and LABELED 'Enabled'"
Will this be part of other ticket (missing translation for 3.0 or such)?
Otherwise its OK.
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-de
stent with other HBAC and SUDO add dialogs?
Otherwise it's OK.
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
On 09/26/2011 08:57 PM, Endi Sukma Dewata wrote:
The IPA.association_adder_dialog has been modified not to show
search results that are already selected to prevent duplicates.
Ticket #1859
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
y
on some metadata/spec information). This all depends on how we want to
make the framework flexible.
--
Petr Vobornik
From ac0fd222072d9360ac50dfd2ea7db9c8cdee6c6c Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Tue, 27 Sep 2011 10:00:27 +0200
Subject: [PATCH] Fixed: Enrolment dialog off
On 09/15/2011 01:51 AM, Endi Sukma Dewata wrote:
On 9/14/2011 7:23 AM, Petr Vobornik wrote:
Forgot to update tests - to address newly added validation row in
table_widget.
One issue, in all search and association facets we now have 2 rows of
footer (there are 2 horizontal lines at the bottom
On 09/28/2011 01:02 AM, Endi Sukma Dewata wrote:
The width of the 1st level tab has been modified to expand according
to the size of the tab label.
The width of the adder dialogs have been increased to allow longer
button labels.
Ticket #1825
ACK
--
Petr Vobornik
On 09/28/2011 12:59 AM, Endi Sukma Dewata wrote:
On 9/16/2011 12:16 PM, Endi Sukma Dewata wrote:
The service type field in the service adder dialog has been modified
to use an editable combobox.
Ticket #1633.
Rebased (removed undo parameter).
ACK
--
Petr Vobornik
probably can be
combined with the IPA.action_button/IPA.button class so we can use icons
or enable/disable buttons in a more consistent way. This is one step in
that direction.
I Agree. This would be nice especially in association table widget.
ACK
--
Petr Vobornik
On 09/27/2011 10:54 PM, Endi Sukma Dewata wrote:
On 9/27/2011 7:50 AM, Petr Vobornik wrote:
I've added padding and unified font-weight in error-message style. The
padding is added because text of the message was right on the border
which wasn't much readable. Font-weight is add
On 10/01/2011 12:09 AM, Endi Sukma Dewata wrote:
The hard-coded 'undo' and 'undo all' labels have been moved into
internal.py to allow translation.
Ticket #1897
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-de
On 10/01/2011 12:10 AM, Endi Sukma Dewata wrote:
The Delete button for sudo options in sudo rule details page now
will only work if there is at least one row selected.
Ticket #1896
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel
https://fedorahosted.org/freeipa/ticket/1922
gidNumber is not an allowed attribute for a non-posix group. When
adding a non-posix group from the UI, unchecking the "Is this a POSIX
group?:" box should disable the "GID:" field.
--
Petr Vobornik
From 3e329f7f6e26cf839681
On 10/04/2011 11:59 PM, Adam Young wrote:
On 10/04/2011 12:43 PM, Petr Vobornik wrote:
https://fedorahosted.org/freeipa/ticket/1922
gidNumber is not an allowed attribute for a non-posix group. When
adding a non-posix group from the UI, unchecking the "Is this a POSIX
group?:"
needs to go to ipa-2-1 branch).
Rebased on top of 292.
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
On 10/01/2011 05:42 AM, Endi Sukma Dewata wrote:
The radio buttons in association facet and radio widget are now
linked to their labels so that they can be selected by clicking
the labels.
Ticket #1782
ACK
--
Petr Vobornik
___
Freeipa-devel
.
Expected results:
Title "Identity Management" is always present.
Fixed:
* modified paths to images
* fixed padding in ssbrowser.html
* moved browser icons to ui folder
* deleted unused images in html and migration folders (they are
already in ui folder, and weren't deployed)
-
was second part of 021, but the file name
was wrong(021-1)
--
Petr Vobornik
From 4785e6bbc9d4254025bb3e06e865f2eccac9f36f Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Mon, 10 Oct 2011 13:34:15 +0200
Subject: [PATCH] Circular entity dependency
https://fedorahosted.org/freeipa/ticket/1531
https://fedorahosted.org/freeipa/ticket/1944
(2.1.3 Release)
No editable fields exist for "maxfail", "failinterval" "lockouttime" and
"priority" in password policy page.
--
Petr Vobornik
From 9ae5eca65de34c02fe0c3baae6eb27e2fa8fe346 Mon Sep 17 00:00:00
https://fedorahosted.org/freeipa/ticket/1987
--
Petr Vobornik
From 931b27dbb54ace65e2213ffed718ee04ace5fc07 Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Mon, 17 Oct 2011 11:48:03 +0200
Subject: [PATCH] Fixed: Unable to add external user for RunAs User for Sudo
rules
https
Comments in text. Also attached git diff for convenience.
On 10/17/2011 10:39 PM, Endi Sukma Dewata wrote:
> On 10/10/2011 10:13 AM, Petr Vobornik wrote:
>> https://fedorahosted.org/freeipa/ticket/1531
>>
>> (3.0 Core Effort Iteration 01 September Y11 Release)
>&
Fix for regression in unit test, introduced by previous patch.
--
Petr Vobornik
From f1a1a99763f38a10304c374da88857c85c9e7748 Mon Sep 17 00:00:00 2001
From: Petr Vobornik
Date: Thu, 20 Oct 2011 16:37:48 +0200
Subject: [PATCH] Fixing infinite loop in UI navigation unit test.
https
you think is not necessary then ACK.
This patch is also fixing https://fedorahosted.org/freeipa/ticket/1973 .
--
Petr Vobornik
<>___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
Comments in text. New patch not supplied yet - some topics may require
further discussion. Most of the comments should be part of the 'Nesting
widgets' thread.
On 10/24/2011 06:29 PM, Endi Sukma Dewata wrote:
On 10/24/2011 3:29 AM, Petr Vobornik wrote:
https://fedorahosted.org/free
On 10/24/2011 10:43 PM, Endi Sukma Dewata wrote:
On 10/21/2011 6:40 AM, Petr Vobornik wrote:
1) Wouldn't be better if the asterisk has different color than the
label? Visually I don't like it that much and I think it can be
overlook. Attaching a proposition. I used green IPAish col
o any harm.
2) Remaining string in internal.py: hbacrule.deny (couldn't find any usage).
Maybe these don't even need to be fixed. (-> ACK on your judgement)
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https:/
On 10/25/2011 10:10 PM, Endi Sukma Dewata wrote:
On 10/25/2011 10:40 AM, Petr Vobornik wrote:
1) Some references remained in testing data: hbacrule_find.json,
hbacrule_show.json. Anyway these don't do any harm.
Fixed.
2) Remaining string in internal.py: hbacrule.deny (couldn'
different language. It should be translated. We
can fix it separately, it is also present in SUDO rules search facet.
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
renamed to entity_adder_dialog but its method
add_dialog_create wasn't renamed
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
On 10/26/2011 07:44 PM, Endi Sukma Dewata wrote:
On 10/26/2011 7:09 AM, Petr Vobornik wrote:
1) add_dialog was renamed to entity_adder_dialog but its method
add_dialog_create wasn't renamed
Fixed.
ACK
--
Petr Vobornik
___
Freeipa-devel ma
On 10/26/2011 07:59 PM, Endi Sukma Dewata wrote:
The metadata and param_info attributes in widget have been merged
because they are redundant.
Ticket #1436
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https
On 10/26/2011 08:09 PM, Endi Sukma Dewata wrote:
The validation code in details facet, dialog, and sections have
been modified to work more consistently.
This is required by patch #301.
ACK
--
Petr Vobornik
___
Freeipa-devel mailing list
Freeipa
On 10/26/2011 08:11 PM, Endi Sukma Dewata wrote:
The user adder dialog has been modified to provide optional fields
to specify password during user creation.
Ticket #1646
1) message in: 'field2.show_error('Passwords do not match');' should
use translated string
1 - 100 of 1861 matches
Mail list logo