search

RedHatSatellite / ballista

Modular and easily extendable tool to manipulate Katello/Satellite 6
GNU General Public License v3.0
16 stars 10 forks source link

Add support for updating non-composite content views #16

Closed cFire closed 4 years ago

cFire commented 4 years ago

I don't have any composite content views to test with in my environment, so can't fully test if that works properly.

yhekma commented 4 years ago

@cFire Do you have a need for this? If you cannot test it, I wonder why the PR? Don't get me wrong, I really appreciate it and I see no obvious problems with it, although I haven't touched Satellite in years to be honest.

cFire commented 4 years ago

Ah, I should explain better;I cannot test if composite content views still work correctly. The change is to allow non-composite content view to be updated. This is what I need, and this I have tested.

yhekma commented 4 years ago

Ah, clear. Well, good to know this thing still gets some use out there :)