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] alexey-pelykh as member of crm #19

Closed
wants to merge 1 commit into from

Conversation

alexey-pelykh
Copy link

No description provided.

@rousseldenis
Copy link
Contributor

@alexey-pelykh Could you point out your different PR's in the repos ? Thanks

@rousseldenis
Copy link
Contributor

I would say your contribution is currently too limited to apply as it covers just one module.

@OCA/crm-sales-marketing-maintainers

@pedrobaeza
Copy link
Member

You can convert into maintainer of such module and have the same merge rights as being a PSC in that module.

@alexey-pelykh
Copy link
Author

@pedrobaeza as you might have guessed, that specific module just happens to be the first one of a few that I'm interested in, and I'm already extremely happy about response time :D obviously, nothing surprising or new :)

@pedrobaeza
Copy link
Member

Yeah, but this PSC is very wide for giving full access. You can review other PRs and ask in exchange that other review yours.

@alexey-pelykh
Copy link
Author

Can't argue with that, indeed withs PSC team covers too much repos

@alexey-pelykh
Copy link
Author

OCA/sale-workflow#3010 then and I'll close off this for now yet it seems that it's also good to split the CRM PSC team into more than one.

@rousseldenis
Copy link
Contributor

OCA/sale-workflow#3010 then and I'll close off this for now yet it seems that it's also good to split the CRM PSC team into more than one.

Maybe, but you found the solution by adopting the module 😃

@simahawk simahawk closed this May 30, 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.

4 participants