-
Notifications
You must be signed in to change notification settings - Fork 94
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
Org referents #2697
Org referents #2697
Conversation
61e0bff
to
2a82c83
Compare
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.
Hi Remi, I've rebased on top of master for you since it was my fault to commit the code formatting with the wrong formatter.xml file.
Please do:
git remote add groldan git@github.com:groldan/georchestra.git
git fetch groldan
git reset --hard groldan/org-referents
Before continuing with the requested changes, which btw are rather minimal, PR looks good overal.
console/src/main/java/org/georchestra/console/ws/editorgdetails/EditOrgDetailsFormBean.java
Outdated
Show resolved
Hide resolved
...le/src/main/java/org/georchestra/console/ws/editorgdetails/EditOrgDetailsFormController.java
Outdated
Show resolved
Hide resolved
.../src/main/java/org/georchestra/console/ws/edituserdetails/EditUserDetailsFormController.java
Outdated
Show resolved
Hide resolved
...le/src/main/java/org/georchestra/console/ws/editorgdetails/EditOrgDetailsFormController.java
Outdated
Show resolved
Hide resolved
...rc/test/java/org/georchestra/console/ws/editorgdetails/EditOrgDetailsFormControllerTest.java
Outdated
Show resolved
Hide resolved
...rc/test/java/org/georchestra/console/ws/editorgdetails/EditOrgDetailsFormControllerTest.java
Outdated
Show resolved
Hide resolved
...rc/test/java/org/georchestra/console/ws/editorgdetails/EditOrgDetailsFormControllerTest.java
Outdated
Show resolved
Hide resolved
...rc/test/java/org/georchestra/console/ws/editorgdetails/EditOrgDetailsFormControllerTest.java
Outdated
Show resolved
Hide resolved
932fc20
to
d0b0bd1
Compare
@@ -29,6 +29,13 @@ editUserDetailsForm.organisation=Organization | |||
editUserDetailsForm.url=Website | |||
editUserDetailsForm.areaOfCompetence=Area of competence | |||
editUserDetailsForm.members=Members | |||
editUserDetailsForm.editOrg=Edit Organization |
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 PR should also update the other translation files.
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.
I'm doing it :-(
Add the ability for a user with the role
REFERENT
to edit some of the field of his organization.REFERENT
role is now a protected role in georchestra. This PR contain the frontend and the bacend implementation.