search

holdex / pr-time-tracker

https://autoinvoice-theta.vercel.app
0 stars 2 forks source link

Problem: missing documentation for analysts (manager) #254

Open zolotokrylin opened 5 months ago

zolotokrylin commented 5 months ago

Existing events PR_APPROVED and PR_REJECTED were updated to comply with our needs, meaning the sender is the owner of the PR and these events should be considered as approve or reject received by that user. The newest events PR_REVIEW_{APPROVE|REJECT|COMMENT} are for reviewers purpose (the sender gave a review).

Originally posted by @georgeciubotaru in https://github.com/holdex/pr-time-tracker/issues/248#issuecomment-2129189566

This needs to be documented in the README. We will forget this soon (I will for sure because it is not very intuitive).

zolotokrylin commented 5 months ago

Problem: the link to analysts (manager) interface is missing