Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Problem: missing documentation for analysts (manager) #254

Open
Tracked by #259
zolotokrylin opened this issue May 24, 2024 · 1 comment
Open
Tracked by #259

Problem: missing documentation for analysts (manager) #254

zolotokrylin opened this issue May 24, 2024 · 1 comment
Labels
type: docs Improvements or additions to documentation

Comments

@zolotokrylin
Copy link
Member

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 #248 (comment)

This needs to be documented in the README.
We will forget this soon (I will for sure because it is not very intuitive).
@zolotokrylin zolotokrylin added the type: docs Improvements or additions to documentation label May 24, 2024
@zolotokrylin
Copy link
Member Author

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

@zolotokrylin zolotokrylin mentioned this issue May 24, 2024
Open
@zolotokrylin zolotokrylin changed the title Problem: meaning documentation for analysts (manager) Problem: missing documentation for analysts (manager) May 24, 2024
This was referenced Jun 3, 2024
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: docs Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@zolotokrylin