Skip to content

Conversation

@adamjenkins
Copy link
Contributor

The Book of Abstracts will include the abstract submitter's email address as Corresponding Author in the generated PDF. This should perhaps be changed so that the default is "none" ensuring any information disclosure is intentional.

https://talk.getindico.io/t/submitted-pull-request-to-change-one-default/4296

@ThiefMaster
Copy link
Member

ThiefMaster commented Aug 3, 2025

For the future, please avoid opening PRs from your master branch (or committing things in there) - create a separate branch instead.

I usually squash+merge PRs, so I will basically mess up your master branch by force-pushing to it when preparing the PR for merge (e.g. adding a changelog entry).

@adamjenkins
Copy link
Contributor Author

My apologies. I'm more of a server admin than a programmer, and as such, Github is kind of foreign territory for me. I don't exactly know what I'm doing here. So, sorry if I get the customs wrong.

Do you need me to close this PR?

@ThiefMaster
Copy link
Member

no, it's ok, it was just a suggestion for the future

@ThiefMaster ThiefMaster added this to the v3.3 milestone Aug 19, 2025
adamjenkins and others added 3 commits August 19, 2025 17:50
The Book of Abstracts will include the abstract submitter's email address as Corresponding Author in the generated PDF. This should perhaps be changed so that the default is "none" ensuring any information disclosure is intentional.
@ThiefMaster ThiefMaster changed the title Update settings.py to make BOA hide emails by default Hide BoA corresponding author emails by default Aug 19, 2025
@ThiefMaster ThiefMaster enabled auto-merge (squash) August 19, 2025 16:01
@ThiefMaster ThiefMaster merged commit 249a903 into indico:master Aug 19, 2025
10 checks passed
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.

3 participants