Merge lp:~jontai/openvista-gtm-integration/bug370653 into lp:openvista-gtm-integration
Proposed by
Jon Tai
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | not available | ||||
Proposed branch: | lp:~jontai/openvista-gtm-integration/bug370653 | ||||
Merge into: | lp:openvista-gtm-integration | ||||
Diff against target: | None lines | ||||
To merge this branch: | bzr merge lp:~jontai/openvista-gtm-integration/bug370653 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
jeff.apple | Approve | ||
Review via email: mp+7867@code.launchpad.net |
To post a comment you must log in.
I don't see anything wrong, though I haven't tested.
One small note: some of the new labels used by the switch are 9 characters long. SAC (2.2.4) only allows for 8. However, since this is a GT.M only module I would think it's OK since GT.M will never have a problem with it.