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

access: superadmin role ID fix #3392

Closed

Conversation

jmacmahon
Copy link

Signed-off-by: Joe MacMahon joe.macmahon@cern.ch

@jmacmahon jmacmahon changed the title access: superadmin role ID fix WIP access: superadmin role ID fix Jul 23, 2015
@jmacmahon jmacmahon changed the title WIP access: superadmin role ID fix access: superadmin role ID fix Jul 23, 2015
@jirikuncar
Copy link
Member

* FIX Sets superadmin role ID included in roles list returned from
  acc_find_possible_roles to the correct, current value.  (closes inveniosoftware#3390)

Signed-off-by: Joe MacMahon <joe.macmahon@cern.ch>
@jmacmahon
Copy link
Author

Done

@jirikuncar jirikuncar self-assigned this Jul 27, 2015
@jirikuncar
Copy link
Member

@jmacmahon can you mention as INCOMPATIBLE that we are removing CFG_SUPERADMINROLE_ID?

Please rebase this branch on top of maint-2.1 and try to keep it up-to-date until merged. Thanks

@jirikuncar jirikuncar assigned jmacmahon and unassigned jirikuncar Jul 27, 2015
jalavik pushed a commit to inspirehep/invenio that referenced this pull request Aug 4, 2015
* FIX Sets superadmin role ID included in roles list returned from
  acc_find_possible_roles to the correct, current value.
  (closes inveniosoftware#3390) (closes inveniosoftware#3392)

* INCOMPATIBLE Removes configuration option CFG_SUPERADMINROLE_ID.

Reviewed-by: Jiri Kuncar <jiri.kuncar@cern.ch>
Signed-off-by: Joe MacMahon <joe.macmahon@cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

access: CFG_SUPERADMINROLE_ID is not accurate in the demosite.
3 participants