Code review comment for lp:~jontai/openvista-gtm-integration/bug370653

Revision history for this message
jeff.apple (jeff-apple) wrote :

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.

review: Approve

« Back to merge proposal