We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
We signal the collaboration server from the viewset, it is not the best place to do it.
Better to move it to the model.
https://github.com/numerique-gouv/impress/blob/7dfc62b2c543064f88be33b7746b8a596633294b/src/backend/core/api/viewsets.py#L525-L526
https://github.com/numerique-gouv/impress/blob/7dfc62b2c543064f88be33b7746b8a596633294b/src/backend/core/api/viewsets.py#L832-L833
https://github.com/numerique-gouv/impress/blob/7dfc62b2c543064f88be33b7746b8a596633294b/src/backend/core/api/viewsets.py#L841-L843
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Problem
We signal the collaboration server from the viewset, it is not the best place to do it.
What to do
Better to move it to the model.
Code location
https://github.com/numerique-gouv/impress/blob/7dfc62b2c543064f88be33b7746b8a596633294b/src/backend/core/api/viewsets.py#L525-L526
https://github.com/numerique-gouv/impress/blob/7dfc62b2c543064f88be33b7746b8a596633294b/src/backend/core/api/viewsets.py#L832-L833
https://github.com/numerique-gouv/impress/blob/7dfc62b2c543064f88be33b7746b8a596633294b/src/backend/core/api/viewsets.py#L841-L843
The text was updated successfully, but these errors were encountered: