refactor to service style
Merge branch 'master' into refactorEmailControler
Fixed error skipping when resaving profiles.
Don't update indexing status when resaving (by default).
refactor find to use findByClientId
modify test and manager to refect live application caching
Removing @PREAMBLE and @COMMENT push
Merge pull request #1232 from ORCID/claimReminderPatch

Fix copy and past cliam reminder email error
Merge branch 'master' into SkipEntryTypes
Skip PREAMBLE and COMMENT, also the the work is deleted from the list only when it was saved, when cancel the job persist in the list.
Merge branch 'master' into SkipEntryTypes
Merge pull request #1227 from ORCID/refactorEmailControler

Refactor email controler
Merge pull request #1229 from ORCID/dontUpdateIndexingStatusWhenResaving

Dont update indexing status when resaving
Merge pull request #1230 from ORCID/moreClientDetailsCaching

refactor find to use findByClientId
Merge pull request #1231 from ORCID/SkipEntryTypes

Removing @PREAMBLE and @COMMENT push
make sure the user has one working email before allowing public client
Formatted
"Sign In" link on "is this you" modal takes you out of the oauth flow
Changed funding external id jaxb -> jpa conversion so that reuses
existing entities where possible.

This was to fix the following error (when de-duplication occurs during
re-saving of the profile).

java.lang.IllegalArgumentException: Removing a detached instance
org.orcid.persistence.jpa.entities.FundingExternalIdentifierEntity#15299
Merge pull request #1233 from ORCID/my-orcid3SecendTry

My orcid3 secend try