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.

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

Release 4.2.0 #127

Merged
merged 44 commits into from
Jan 21, 2022
Merged

Release 4.2.0 #127

merged 44 commits into from
Jan 21, 2022

Conversation

Baggerone
Copy link
Contributor

@Baggerone Baggerone commented Jan 19, 2022

Added

  • Added "secureuser" ID Store adapter
  • HR Notification on new user creation, supported by Workday and Google Sheets ID Stores

Fixed

  • Scrutinizer config change, only fail for bugs and security issues

forevermatt and others added 30 commits January 2, 2018 13:53
groundwork for new notification feature
…cation

send new user notification email if enabled
…-useful

Only fail Scrutinizer check for bugs/security issues.
…email

check to be sure HR Contact Email is not empty
Comment on lines +37 to +39
/**
* {@inheritdoc}
*/
Copy link
Contributor

Choose a reason for hiding this comment

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

An otherwise-empty doc. block that just contains {@inheritdoc} can simply be left off, as good IDEs will automatically bring in the inherited docs.

application/common/config/main.php Show resolved Hide resolved
application/common/models/User.php Show resolved Hide resolved
@Baggerone Baggerone merged commit 1adb746 into main Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants