Bug 454593
Summary: | change hwcert code to work with bugzilla 3.2 after NEEDINFO status cleanup | ||
---|---|---|---|
Product: | [Retired] Red Hat Hardware Certification Program | Reporter: | Noura El hawary <nelhawar> |
Component: | Hardware Catalog | Assignee: | Rob Landry <rlandry> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Yu Shao <yshao> |
Severity: | high | Docs Contact: | |
Priority: | high | ||
Version: | 5 | CC: | bxu, efeng, rlandry, xiqin, xisun, ykun, yshao |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | 3.2rh-20080801.1 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2008-08-04 17:57:04 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 406071, 437069, 443456 | ||
Attachments: |
Description
Noura El hawary
2008-07-09 07:07:44 UTC
Rob, just wanted to touch base that the necessary information has been provided and you guys will be ready to go with this by August 2nd along with your other 3.2 related changes? Thanks Dave Dave, I believe we have the information needed, this has been assigned and Eric is working on a patch. I'll talk with him tonight to find out when he plans on having the patch complete. We are still targeting the 7/26 date in hwcert to ensure we have the extra week as a window. -Rob Created attachment 312231 [details]
But to map the needinfo to NEEDINFO
Created attachment 312356 [details] patch to fix (1)-(5) entries in comment#5 patch to fix (1)-(5) entries in comment#5 Created attachment 312360 [details]
a patch have been cleaned some useless thing out
Created attachment 312381 [details] newest first-part patch which fix the suggestion in comment#8 and comment#10 :) I have already uploaded the newest first-part patch which fix the suggestion in comment#8 and comment#10 :) edit.html: fine status_edit.html.tmpl @@ -2,26 +2,12 @@ The </hr> should remain, the status_edit template is currently correct, rh_edit (has an </hr> outside of an [% IF %]) and tam_ack (missing the starting </hr>) templates have errors which cause this. @@ -49,8 +35,9 @@ newstatus will need another default other than needinfo on the '<input type="hidden" name="newstatus" value="needinfo"' line General, because the drop down is removed we should change to using all caps for status so the 4 states are: Open: Leave as IN PROGRESS Change status to CLOSED/CERTIFIED Closed: Leave as CLOSED/CERTIFIED Re-Open certification as IN PROGRESS Sir I have uploaded a new the patch which fixed above Created attachment 312413 [details]
fixed above comments
please review @ http://bugdev.devel.redhat.com/hwcert-eric2/show.cgi?id=456123 Looks good, I'm not certain the rh_edit @@ -4,7 +4,7 @@ change is really needed but otherwise please commit this change. Created attachment 312693 [details] Updated needinfo as a flag patch Replaces previous efforts, uses the existing flags infrastructure from within a new template; also address a User.pm change where email_prefs was removed in the bz3 effort but is required by the flags module; also corrects for a missing use of the is_active field deprecated in bug #443465 disguised as "exists". Unfortunately the db is down for now so I can no longer verify function, so it'll probably still need some UI updates though @ last run before the test db's went down for maint. basic operation was correct, but the JS to auto select the radio's wasn't working. The patch has been cleaned up since them to simplify the interface and may no longer work, or work as expect but not as desired or some combination. Created attachment 312921 [details]
minor update to needinfo-flag patch
Changes committed to CVS
testedd and run good |