-
Notifications
You must be signed in to change notification settings - Fork 129
Associate email address with organization for author field on new commits #95
Comments
This comment has been minimized.
This comment has been minimized.
2 similar comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
I have exactly this issue, would be great to see this changed. |
This comment has been minimized.
This comment has been minimized.
17 similar comments
👍 |
👍 |
+1 |
+1 |
👍 |
+1 |
+1 |
+1 |
+1 |
+1 |
+1 |
+1 |
+1 |
👍 |
👍 |
+1 |
👍 |
Any news on when (and if) this will be implemented? |
This comment has been minimized.
This comment has been minimized.
2 similar comments
+1 |
👍 |
I emailed [email protected] about this. Here is their response:
|
wow, I wrote this request over two years ago. Thanks for the +1 collection everyone :-/ |
This comment has been minimized.
This comment has been minimized.
1 similar comment
👍 |
This is a very serious issue for me, also because I'm part of two organizations and my personal email is being attached to stuff that is not even my property. |
Unfortunately it isn't possible to fix github commit with this approach, is it? Looks like you just replace the committer email before passing it to external system (like |
+1 |
We're considering using GitHub more than we do now because Bitbucket has a huge problem: they use an email for a login identifier, which ties the account to some company and causes all sorts of problems. GitHub seems to have a better approach: a single login ID, independent from any company, and I can add as many emails as I like. Great. But now it seems that GitHub still ties me to a single address for many web-based commits, e.g. merging pull requests? I can't associate emails with organizations, so that when I merge a pull request for a particular organization, it always uses the correct email? That's a bummer, and suddenly puts me back in the same boat as Bitbucket. Please let us associate web-based commit emails with different repositories and different organizations. Thank you. |
Facing the same exact issue with my team |
I think https://github.blog/changelog/2019-03-25-view-organization-members-email-addresses/ might be considered progress toward this, as it seems like work toward being part of the infrastructure necessary to implement it. |
You can now specify your git commit email when merging a PR!! |
This is a great step, thanks! I think ideally it would be nice to be able to set the email address on a per-project basis, so that the merge requests would automatically default to the correct email address. This is not an outlandish idea; we can already do the same thing locally on a per-repository basis using the following Git command (leaving off
But this new option is a great intermediate step. Thanks again for implementing it. |
@garretwilson the drop down will default to the last email you used to commit to the repo, defaulting to your organization notification email if this is your first commit, meaning ideally, it should default to the email you'd like to use for the repo, or will after your first use, without needing to manage an additional setting. |
This is awesome! |
Thanks for the all the feedback! If you have concerns or find bugs leave them here and contact support. If there are followup features you'd like to see please open a new issue and tag this one. 🚢 https://github.blog/changelog/2019-04-24-choose-your-merge-commit-email/ |
Thank you for doing this! |
It seems to always be defaulting to my first email, I'm having to change it every time. If I push a commit directly (aka not through a PR), is it associating to my first email instead and resetting this? Would be nice if there was a way to just configure this for my repo and force a value to be the default |
After I suggested that (noting that this is already available in Git locally), I realized that the more important thing would be to associated the email with the organization itself, or at least specify a default for each organization. And now I realize that this is exactly what this ticket originally requested! 😆 So good work, but let's not forget semantics. Thank you! |
Thank you for this! Makes working for several different organisations much easier without the need to change global settings all the time. |
@DrLeh if you commit to a repository via any means (make a commit locally and push to the repo on GitHub, or make a commit through the online editor), the email used to make that commit will be the one that these features default to next time. Hope this helps! If you're still experiencing this issue, please feel free to reach out to GitHub Support so we can investigate this further =] |
@gallexi yeah it's not working for me. I have two emails configured A and B
|
@DrLeh Had a chance to dig into this a bit more and wanted to share two scenarios where I believe the drop down may not reflect the last email used in the drop down:
The reason for both is that we look for the most recent commit, on any branch, that you authored. If you've authored other commits since the last merge via the web interface using another email, those commits will override the preference. Similarly, if you squash merge another user's pull request, the actor is set as the committer and the PR author is set as the commit author (for proper attribution), in which case the email you selected via the dropdown for that squash merge (of another user's pull request) would not be preserved and it would default to your most recent authored commit or your per-organization notification email, if set. /cx todogroup/gh-issues#32 for a similar discussion. |
@benbalter I figured that might be the case. I tried doing a merge commit instead of a squash and that fixed it, I didn't get a chance to respond here yet. Thanks! |
Please can we have it default to the organisation's email for squash merges too? I have a scenario where I'm the only committer of several commits in a PR and when I go to squash, it's still not defaulting to the correct default email for the organisation (even though this is the email used for each commit in the log). |
This doesn't work when doing a (squash) merge via the API |
I don't see the email dropdown anymore when trying yo squash or merge commit a PR! I am sure it was there a few weeks ago, but now I am not prompted with the drop down anymore. I am committing PRs that I have authored with my organisation address, but they now appear with my personal address when squashed. |
Will try to repro but I am fairly certain my github UI keeps defaulting back to my main (personal) email. At work I merge PRs 5-10 times/week, I have been doing no side-projects at all lately so I know I've never set it back to my personal email. I have definitely set it to my work email (upon clicking "squash and merge") several times and never set it back to my personal email. I keep getting personal email selected. |
I love the “custom routing” feature in https://github.com/settings/notifications , which allows me to configure notification email addresses depending on which organization owns a repository.
The problem I'm having is that this is restricted to outgoing notification emails. I'd like to take this association farther. Specifically, to the author email on commits generated via Github's browser interface.
Our workflow involves merging story branches into master (or other long-running branches) inside Github, when the merge button is green. If I do this however, the merge commit doesn't get my email address for that organization. Instead, it gets my primary email address. This affects other applications integrated with git/github. For example, it affects Travis CI, which sends build success or failure emails to my personal email address instead of my organization email address.
The text was updated successfully, but these errors were encountered: