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.

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

[ADD] github_connector_pull_request #110

Open
wants to merge 1 commit into
base: 14.0
Choose a base branch
from

Conversation

SirAionTech
Copy link
Contributor

No description provided.

@SirAionTech SirAionTech force-pushed the 14.0-add-github_connector_pull_request branch from 382a0b7 to bee4106 Compare September 26, 2023 14:26
@SirAionTech SirAionTech marked this pull request as ready for review September 26, 2023 14:31
Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice work ! code review. No test.
I wonder if we should add a cron to synchronize PR regularly.

like in the core module : https://github.com/OCA/interface-github/blob/14.0/github_connector/data/ir_cron.xml

@SirAionTech SirAionTech force-pushed the 14.0-add-github_connector_pull_request branch from bee4106 to 695d534 Compare September 27, 2023 07:43
@SirAionTech
Copy link
Contributor Author

nice work ! code review. No test.
I wonder if we should add a cron to synchronize PR regularly.

like in the core module : https://github.com/OCA/interface-github/blob/14.0/github_connector/data/ir_cron.xml

Thanks for your review!
I was looking forward to your feedback specifically because you implemented the core modules.
I added the CRON

Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Design question : Why create a dedicated module.

  • If we install github_connector, we expect to have the possibility to fetch also the PRs, not only the branches, and the code ?
  • if then, we want to implement "github_connector_issue", it will ask a refactor, because a lot of thing are common between issues and pull requests, so we should create an fourth module "abstract".
  • the PR / issues improvment are written in the roadmap of the main module : https://github.com/OCA/interface-github/tree/14.0/github_connector#known-issues-roadmap

Proposing to improve the existing module. What do you think ?

Question : did you thought to fetch also comments ?

note : I began to work on PR / issue / comment in V8. If it can help :
https://github.com/OCA/interface-github/pull/1/files#diff-a1b4664848d92d0cabbe8108e17e7480403c09afbb1ee07242878b4273f3d564R18

@SirAionTech
Copy link
Contributor Author

  • If we install github_connector, we expect to have the possibility to fetch also the PRs, not only the branches, and the code ?
    ...

Proposing to improve the existing module. What do you think ?

I see Pull Requests as a temporary object, I think most users should only be interested in the main branches.
The ordinary user might then be not interested in the Pull Requests, so it should not install the additional module.

We implemented this module because we are also interested in what is the work in progress of some repositories.

did you thought to fetch also comments ?

No

Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jan 28, 2024
@SirAionTech
Copy link
Contributor Author

Any news on this one?

@legalsylvain
Copy link
Contributor

I'm abstaining on this one!

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 4, 2024
Copy link

github-actions bot commented Jun 9, 2024

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 9, 2024
Co-authored-by: gabriele-baldessari <[email protected]>
@SirAionTech SirAionTech force-pushed the 14.0-add-github_connector_pull_request branch from 695d534 to 887a640 Compare June 10, 2024 07:15
@SirAionTech
Copy link
Contributor Author

Rebased on 14.0.

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants