Comment 3 for bug 1234927

Revision history for this message
Mike Rylander (mrylander) wrote :

Chris,

I have one request that impacts consistency and maintainability. Please pluralize the field name (and, as a consequence, the link name) to match the existing "circulations" field. Because IDL link fields are available in the code, and not just behind the scenes in the reporter, we need to follow the pluralization conventions everywhere.

Otherwise, looks good.

Thanks!