-
Notifications
You must be signed in to change notification settings - Fork 56
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
Object store and packet writer implementation #13
Conversation
… tags, get tags added
//@Data | ||
//@NoArgsConstructor | ||
//@AllArgsConstructor | ||
public class QualityType implements Serializable { |
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.
All the classes related to BIR should be part of packetmanager
|
||
//@XmlType(name = "PurposeType") | ||
//@XmlEnum | ||
public enum PurposeType implements Serializable { |
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.
All the classes related to BIR should be part of packetmanager
|
||
//@XmlType(name = "ProcessedLevelType") | ||
//@XmlEnum | ||
public enum ProcessedLevelType implements Serializable { |
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.
All the classes related to BIR should be part of packetmanager
@@ -0,0 +1,117 @@ | |||
package io.mosip.registration.clientmanager.contant; | |||
|
|||
public class PacketManagerConstants { |
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.
why is the packetManagerConstants added in clientmanager ?
|
||
//@XmlType(name = "SingleTypeType") | ||
//@XmlEnum | ||
public enum BiometricType implements Serializable { |
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.
All the classes related to BIR should be part of packetmanager
@@ -28,6 +28,17 @@ android { | |||
} | |||
|
|||
dependencies { | |||
implementation 'com.google.dagger:dagger:2.38.1' |
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.
dagger, lombok are already added, please remove duplicates
@Data | ||
@AllArgsConstructor | ||
@NoArgsConstructor | ||
public class CryptoRequestDto { |
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.
This DTO is already present in keymanager
@Data | ||
@AllArgsConstructor | ||
@NoArgsConstructor | ||
public class CryptoResponseDto { |
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.
This DTO is already present in keymanager
@AllArgsConstructor | ||
@NoArgsConstructor | ||
public class PublicKeyRequestDto { | ||
@NotBlank(message = "Invalid Request") |
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.
This DTO is already present in keymanager
@Data | ||
@AllArgsConstructor | ||
@NoArgsConstructor | ||
public class PublicKeyResponseDto { |
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.
This DTO is already present in keymanager
@Data | ||
@AllArgsConstructor | ||
@NoArgsConstructor | ||
public class SignRequestDto { |
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.
This DTO is already present in keymanager
@Data | ||
@AllArgsConstructor | ||
@NoArgsConstructor | ||
public class SignResponseDto { |
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.
This DTO is already present in keymanager
@Data | ||
@AllArgsConstructor | ||
@NoArgsConstructor | ||
public class SignVerifyRequestDto { |
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.
This DTO is already present in keymanager
@Data | ||
@AllArgsConstructor | ||
@NoArgsConstructor | ||
public class SignVerifyResponseDto { |
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.
This DTO is already present in keymanager
Different PR is raised for the same code changes. |
#2 Port 'Packet Manager' and 'KeyManager' service to new repo
Added PosixImplementation :
Added PackerWriterImplementation.
Pending things