sarthaksin1857 commented on PR #131:
URL: https://github.com/apache/otava/pull/131#issuecomment-4274066073

   @Gerrrr sorry for the wait this is ready for another pass. 
   
   I made it so 
   
   - All optional fields are cleaned
   - the parse_datetime utility handles string, int, long, and datetime inputs 
   - Updated the documentation
   - Added a few tests
   
   I tried to extend the integration tests but it didn't seem to play well with 
tags. Whenever I tried sending an event, it would time out (I can file an issue 
for adding integration coverage and tackle that next?)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to