C+ review for Migrate from reportAction timestamp to created

[mountiny]

This is to track payment to @mananjadhav for reviewing this PR #12540 (comment) which is linked to this internal issue.

[melvin-bot]

Triggered auto assignment to @isabelastisser (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[mountiny]

@mananjadhav what is an appropriate payment for that review and testing please?

[mananjadhav]

@vitHoracek I think this requires a bump because this was a relatively large PR, with 20 files and also finding if the old timestamp instances are left out.

But I would also like to highlight there was a test failure in this issue. (though when I tested I think it did run well don't remember exactly). I am not sure if that counts as a regression. If yes, then the current internal rate should be fine.

[mountiny]

But I would also like to highlight there was a test failure in this issue. (though when I tested I think it did run well don't remember exactly).

Oh I remember that failure and that is not regression, that was not failing during the review but showed up later.

Apologies fo rnot being clear these processes are updated quite often but I found a Slack post here https://expensify.slack.com/archives/C01GTK53T8Q/p1668726522584349 which mentions the internal PRs are reviewed for flat rate of $1000. Is that good for you? otherwise we would need to start a discussion in #contrbutor-plus, we need to make sure these things have rules which can be followed.

[mananjadhav]

I think 1000$ is fine, because I didn't complete the Android Push notification test for the task.

Thanks for clarifying. I think I am going to clarify this. I am posting a question on the C+ channel for the same.

[mananjadhav]

Based on the discussion in the thread, good with the default payout of 1000$

[mountiny]

Thanks for raising this @mananjadhav Never be afraid of raising any questions 🙇