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.

Sign up for GitHub

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

Jump to bottom

requests: manage sending notifications #1877

Merged
merged 1 commit into from
Nov 21, 2024
Merged

requests: manage sending notifications #1877

merged 1 commit into from
Nov 21, 2024

Conversation

anikachurilova
Copy link
Member

alejandromumo
alejandromumo approved these changes Nov 14, 2024
View reviewed changes
Copy link
Member

@alejandromumo alejandromumo left a comment

Choose a reason for hiding this comment

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

LGTM, tests are failing because the linked PR from invenio-request has to be released first

@anikachurilova anikachurilova force-pushed the community-auto-submit-remove-notification-about-request-auto-acceptance branch 5 times, most recently from e6101e3 to 6b5c70e Compare November 19, 2024 09:48
@anikachurilova anikachurilova force-pushed the community-auto-submit-remove-notification-about-request-auto-acceptance branch from 5f8138e to 8760044 Compare November 19, 2024 09:49
@slint slint merged commit 8d2823f into inveniosoftware:master Nov 21, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alejandromumo alejandromumo alejandromumo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

community auto-submit: remove notification about request auto-acceptance
3 participants
@anikachurilova @alejandromumo @slint