-
Notifications
You must be signed in to change notification settings - Fork 15
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
Gdpr notifications #48
base: master
Are you sure you want to change the base?
Conversation
@@ -0,0 +1,64 @@ | |||
package org.gbif.registry.cli.dataprivacynotification; | |||
|
|||
import org.gbif.common.messaging.AbstractMessageCallback; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unused
package org.gbif.registry.persistence.mapper; | ||
|
||
import java.util.List; | ||
import java.util.Map; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unused
import static org.gbif.common.messaging.api.messages.DataPrivacyNotificationMessage.EntityType; | ||
|
||
/** | ||
* Provides convertion from the key value pairs to HStore. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
conversion or transformation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only tiny observations
changes applied |
Steps needed to release these changes:
Pending to add the email text that we want to send to the template ( Note that currently there is a cli version deployed in DEV that sends emails to my gbif email (it's hardcoded). |
13d9076
to
5b30c27
Compare
depends on gbif/postal-service#2