Fix analytics to work with Django ViewSet actions

[aangelinali]

Key Change

• Added @wraps(func) to record_api_function and record_function methods

Explanation for Error
When implementing analytics on Penn Clubs, the following error was thrown.

AssertionError: Expected function (wrapped_func) to match its attribute name (email_blast)

• The reason for this error is that in a ViewSet, Django finds custom endpoints with @action. After inspecting each extra action, Django asserts that the callable's name matches the attribute name on the class. But, the analytics decorator (as it currently is written) returns a wrapper named wrapped_func and raised the error.

Explanation for Fix

• Adding @wraps(func) to the inner wrapper in both functions preserves the original function's metadata (notably, name in this case).
• The reason this error was not encountered when implementing Penn Mobile analytics since Penn Mobile uses APIView / ListAPIView classes. Django does not check the callable's name for APIView methods so the fact that the decorator returns wrapped_func does not raise an error.

[dr-Jess]

Will approve after small nit is fixed so that ci/cd works.

[dr-Jess]

lgtm