#207 Make admin portal easier to discover
Closed: complete 4 years ago by jflory7. Opened 5 years ago by jflory7.

Summary

Change admin portal URL to more obvious path and add link in side menu to admin settings for admin users

Background

This URL was chosen by the upstream project and likely tries to follow security by obscurity. Since Fedora Happiness Packets has an authentication mechanism in place to a secure authentication service (i.e. FAS), obscurity makes it harder for admin users to do their job.

Details

Completing this ticket involves two tasks:

  1. Change URL string to more intuitive URL (i.e. fhp.example.com/drunken-octo-lama/ to fhp.example.com/admin/)
  2. If a user is an admin user, show a link in the side menu to access admin settings page

Outcome

  1. Easier for humans to remember
  2. Makes admin functionality easy to discover for first-time users

Metadata Update from @jflory7:
- Issue marked as depending on: #157
- Issue tagged with: blocked

5 years ago

Metadata Update from @jflory7:
- Issue untagged with: help wanted
- Issue priority set to: waiting on external (was: waiting on assignee)

5 years ago

Hey @jflory7 ! If I am getting this right, you mean we want to have 2 groups, one admin and another user. This ticket aims to provide easy access to admin and some functionality that an admin can have?

@alishapapun Right, there are multiple regular users and possibly multiple admin users. Some of the discussion about this was in #157 starting here.

This ticket adds no new functionality, but makes the admin portal easier to discover when an admin user is logged in. For example, in the drop-down menu with My Received Packets and My Sent Packets, there might be a new link there, Admin settings, when an admin user is logged in.

Does this make sense?

hello @ jflory7, i would like to work on it

Metadata Update from @jflory7:
- Issue marked as depending on: #157

5 years ago

Hi @saidat, sure. How are you thinking to implement this?

Iam going to create a Python module informally called a URLconf (URL configuration). This module is pure Python code and it will be a mapping between URL path expressions to Python functions. The value of the url config will be used in place of the ROOT_URLCONF setting. That's what i think

@alishapapun Right, there are multiple regular users and possibly multiple admin users. Some of the discussion about this was in #157 starting here.
This ticket adds no new functionality but makes the admin portal easier to discover when an admin user is logged in. For example, in the drop-down menu with My Received Packets and My Sent Packets, there might be a new link there, Admin settings, when an admin user is logged in.
Does this make sense?

Absolutely! That makes sense as well. But how are we planning to implement since fedora-happiness-packet authenticate using ipsilon which according to me does not provide any specific facility to any fas-account holders (all are treated as users)?

@alishapapun Right, there are multiple regular users and possibly multiple admin users. Some of the discussion about this was in #157 starting here.
This ticket adds no new functionality but makes the admin portal easier to discover when an admin user is logged in. For example, in the drop-down menu with My Received Packets and My Sent Packets, there might be a new link there, Admin settings, when an admin user is logged in.
Does this make sense?

The task seems quite interesting to work on though. I would like to volunteer to take up the task.

For implementation, in base.html, under the section if user.is_authenticated we need to check whether the current logged-in user is a superuser using this. If yes, using Admin Reverse URLs for fetching the URL to the Django Admin SIte, we need to add a new list item for accessing the Admin Portal.

Hope this helps! :)

Thanks @shraddhaag. @alishapapun, does this answer your question? The changes in PR #214 may be useful to review too.

Metadata Update from @alishapapun:
- Issue assigned to alishapapun

4 years ago

Metadata Update from @jflory7:
- Issue set to the milestone: Summer Coding 2019: phase 1

4 years ago

Metadata Update from @jflory7:
- Issue priority set to: waiting on assignee (was: waiting on external)

4 years ago

Metadata Update from @jflory7:
- Issue untagged with: blocked

4 years ago

And PR merged. :smile: Closing as complete!

Metadata Update from @jflory7:
- Issue close_status updated to: complete
- Issue status updated to: Closed (was: Open)

4 years ago

Login to comment on this ticket.

Metadata