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] gitlab module #311

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

Conversation

ByteMeAsap
Copy link

This module does not provide a functionality by itself but a base model to implement gitlab related features

Comment on lines +31 to +32
This module does not provide a functionality by itself but a base model
to implement gitlab related features
Copy link
Contributor

Choose a reason for hiding this comment

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

this looks very generic, and quite hard to understand the perimeter of this module.
Could you mention links to extra modules that depends on this module ?

thanks !

Copy link
Author

Choose a reason for hiding this comment

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

@legalsylvain ,the extra module is not yet added as its dependent on this module.The module is aimed at recording user activities like commits, pull requests (merge requests), opening issues etc.

Copy link
Contributor

Choose a reason for hiding this comment

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

Hi maybe we could share concept and code with https://github.com/OCA/interface-github module, dont you think ?
With some refactor it looks me possible at a first sight

Copy link
Author

@ByteMeAsap ByteMeAsap Oct 8, 2024

Choose a reason for hiding this comment

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

Here is the link to the PR for the extra module that depends on this module - OCA/vertical-association#178. Actually, its not identical to the interface-github

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