airavata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (AIRAVATA-2571) Have 'email verified' appear in Admin dashboard -->Users
Date Thu, 04 Jan 2018 20:58:00 GMT

    [ https://issues.apache.org/jira/browse/AIRAVATA-2571?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16312018#comment-16312018
] 

ASF GitHub Bot commented on AIRAVATA-2571:
------------------------------------------

machristie closed pull request #80: [AIRAVATA-2571] Have 'email verified' appear in Admin
dashboard -->Users
URL: https://github.com/apache/airavata-php-gateway/pull/80
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/.gitignore b/.gitignore
index 9ab27139f..8d27e80d1 100755
--- a/.gitignore
+++ b/.gitignore
@@ -6,6 +6,12 @@ composer.lock
 .env.php
 .DS_Store
 Thumbs.db
+pga_config.php
 
 /app/storage
-/themes
\ No newline at end of file
+/themes
+
+.idea
+.iws
+workspace.xml
+tasks.xml
diff --git a/app/libraries/Keycloak/Keycloak.php b/app/libraries/Keycloak/Keycloak.php
index 7c4d90c11..5315c6e76 100644
--- a/app/libraries/Keycloak/Keycloak.php
+++ b/app/libraries/Keycloak/Keycloak.php
@@ -259,7 +259,7 @@ public function listUsers(){
         $users = $this->users->getUsers($this->realm);
         $usernames = [];
         foreach ($users as $user) {
-            array_push($usernames, (object)["firstName"=>$user->firstName,"lastName"=>$user->lastName,"email"=>$user->email,"userName"=>$user->username]);
+            array_push($usernames, (object)["firstName"=>$user->firstName,"lastName"=>$user->lastName,"email"=>$user->email,"emailVerified"=>$user->emailVerified,"userName"=>$user->username]);
         }
         return $usernames;
     }
diff --git a/app/views/admin/manage-users.blade.php b/app/views/admin/manage-users.blade.php
index 1b038cb64..166ca0f79 100644
--- a/app/views/admin/manage-users.blade.php
+++ b/app/views/admin/manage-users.blade.php
@@ -58,6 +58,7 @@
                         <th>Last Name</th>
                         <th>Username</th>
                         <th>Email</th>
+                        <th>Email Verified</th>
 
                         <th>
                             Role :
@@ -85,6 +86,11 @@
                         <td>{{ $user->lastName }}</td>
                         <td>{{ $user->userName }}</td>
                         <td>{{ $user->email }}</td>
+                        @if($user->emailVerified)
+                            <td class="text-success"><span class="glyphicon glyphicon-ok"></span></td>
+                        @else
+                            <td class="text-danger"><span class="glyphicon glyphicon-remove"></span></td>
+                        @endif
                         <td>
                             <button class="button btn btn-default check-roles" type="button"
                                     data-username="{{$user->userName}}">Check All Roles


 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Have 'email verified' appear in Admin dashboard -->Users
> --------------------------------------------------------
>
>                 Key: AIRAVATA-2571
>                 URL: https://issues.apache.org/jira/browse/AIRAVATA-2571
>             Project: Airavata
>          Issue Type: New Feature
>          Components: PGA PHP Web Gateway
>    Affects Versions: 0.18
>            Reporter: Eroma
>            Assignee: Neeraj Lad
>             Fix For: 0.18
>
>
> Currently in Admin Dashboard --> Users all the users are listed. We could see the
username and can search using username, first name, last name and email.
> The new requirement is to have email verified also appear for each user. This helps gateway
admins to troubleshoot when users inform gateway access issues.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message