Skip to content
This repository has been archived by the owner on Dec 10, 2023. It is now read-only.

[JENKINS-62291] Update email property as it supposed be done #66

Merged
merged 3 commits into from
Apr 26, 2022

Conversation

DuMaM
Copy link
Contributor

@DuMaM DuMaM commented Apr 22, 2022

This PR should solve all update problems with emails.

There was no proper solution to set a Jenkins Mail task property on Jenkins site.
After fixing some problems with email resolver i noticed that there is nothing what sets those values.
To do so I started to look after already implemented solutions and I found one in AD plugin.

This should fix:
JENKINS-62291
JENKINS-53677
JENKINS-27682 (Maybe)

Relevant changes:
#65

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@DuMaM DuMaM changed the title [JENKINS-62291] Update emails as it supposed to [JENKINS-62291] Update email property as it supposed be done Apr 22, 2022
@DuMaM DuMaM force-pushed the bugfix/JENKINS-62291 branch from 6c2eb89 to 7d24d5a Compare April 25, 2022 16:56
@DuMaM DuMaM force-pushed the bugfix/JENKINS-62291 branch from 31676ca to 5dcdfad Compare April 26, 2022 19:37
@DuMaM DuMaM merged commit d749732 into master Apr 26, 2022
@DuMaM DuMaM deleted the bugfix/JENKINS-62291 branch April 26, 2022 19:52
@DuMaM
Copy link
Contributor Author

DuMaM commented May 11, 2022

After fixes form #72 it also works for Java 11

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant