Report request function

[kylepolich]

Write this function:

from datetime import datetime
import pandas as pd

def get_report(harvest_key: str, start: datetime, end: datetime):
  ...
  df = pd.DataFrame(?)
  return df

This data should have full grain data with as much detail as we can get from the API. We will do GROUP BY later.

Take the time to consider if there are missing requirements. For example, what if a task begins before midnight and ends 25 hours later. Would we forget to query it?

[alexander621]

test project link: https://github.com/mark-uruymagov/harvest-integration

. What do you mean with "harvest_key" param in the "get_report" method?

. In Harvest, they have only "from" and "to" parameters as options and those parameters are by date only. So in the "get_report" method, "start" and "end" params should be "Date" object.
While we are not able to use the parameters to narrow down our call to a timeframe smaller than the date, the time entry objects should be returned with "created at" and "spent at" attributes.
When we've listed the time entries and we have the "created at" and "spent at" attributes, we may be able to use the tool that we're pulling the time entries into to continue to narrow down the entries based on those attributes.

. "What if a task begins before midnight and ends 25 hours later. Would we forget to query it?"
Could you provide some more detail about your concern here? I wanna make sure I am on the same page. Is it that you're polling for a 24-hour timeframe and you might miss an entry that logs more than those 24 hours?

I am still working on it.

[kylepolich]

Hi @mark-uruymagov how is it going?

I see you shared a test link. I am glad to see there is progress, but I won't do any review until it is merged into our repository. Please check the description of the issue and make sure you can do the integration this way, otherwise, it is difficult to test.

[alexander621]

Hi, Kyle.
How are you?

Yes, Got it.

Good night.

[kylepolich]

@mark-uruymagov can you please update your estimate? I had communicated Jan 9th to leadership. We need to give them a revision. Please let me know your estimate.

[kylepolich]

@mark-uruymagov , I'm sorry, but it is being taken out of my hands. The project was meant to be a pilot and the leadership has decided that we are not going to move forward.