-
Notifications
You must be signed in to change notification settings - Fork 4
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
Metalnx fixes 1 #49
Metalnx fixes 1 #49
Conversation
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.
The last commit says "in progress". Is there more still to do for this?
emc-metalnx-services/src/main/java/com/emc/metalnx/services/irods/UploadServiceImpl.java
Outdated
Show resolved
Hide resolved
emc-metalnx-services/src/main/java/com/emc/metalnx/services/irods/PermissionsServiceImpl.java
Outdated
Show resolved
Hide resolved
emc-metalnx-services/src/main/java/com/emc/metalnx/services/irods/PermissionsServiceImpl.java
Outdated
Show resolved
Hide resolved
emc-metalnx-services/src/main/java/com/emc/metalnx/services/irods/PermissionsServiceImpl.java
Show resolved
Hide resolved
emc-metalnx-core/src/main/java/com/emc/metalnx/core/domain/entity/enums/DataGridPermType.java
Show resolved
Hide resolved
...metalnx-services/src/main/java/com/emc/metalnx/services/configuration/ConfigServiceImpl.java
Show resolved
Hide resolved
emc-metalnx-services/src/main/java/com/emc/metalnx/services/irods/PermissionsServiceImpl.java
Outdated
Show resolved
Hide resolved
emc-metalnx-services/src/main/java/com/emc/metalnx/services/irods/PermissionsServiceImpl.java
Outdated
Show resolved
Hide resolved
emc-metalnx-services/src/main/java/com/emc/metalnx/services/irods/UploadServiceImpl.java
Outdated
Show resolved
Hide resolved
emc-metalnx-services/src/main/java/com/emc/metalnx/services/irods/UserServiceImpl.java
Show resolved
Hide resolved
Other dependencies could be updated but they won't be at this time. I will update the commit message. |
588f2b9
to
959876a
Compare
Commits have been squashed. |
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.
Looks good to me. Probably should wait for @korydraughn's approval before #'ing.
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.
Pound it.
…() order to default user to RODS_USER
959876a
to
9159494
Compare
done |
These are changes needed for Metalnx.