odoo / odoo/runbot

Add *teams* sync to the mergebot

Open
#901 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

mergebot
Dominant language
Python
Stars
159
Forks
156
Avg merge
1d 13h
Merged PRs (30d)
16

Description

Currently the mergebot is able to sync employees in order to auto-provision their account & give them backend access.

In order to better classify users (mostly in the context of forward ports) teams were created (as companies), however assignment of employees to those teams / companies has been extremely ad-hoc and I've no really kept up with it.

It would be nice to have accounts automatically push this information to the mergebot, alongside changes in employees (as in, an employee changing team). Although it's not necessarily a super important thing as we can reasonably easily massage import / export features into doing that...

Also would probably make sense to store the reviewer's team on the PR at review (also author's?) such that the PR is then associated specifically with a review team, and the forward ports can more easily be reassigned if the employee changes team or leaves.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No files or tests are named. Start by tracing the existing mergebot employee synchronization and the import/export paths mentioned in the issue, then map how teams and pull-request reviews are represented. Done should include automatic team assignment updates and a defined association between reviewers or authors and the team on each pull request.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
28/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.