backoffice[authors]: show all tickets

* ref: cern-sis/issues-inspire#620
DonHaul · Mar 6, 2025 · 8dc66bc · 8dc66bc
1 parent 78fbb7f
commit 8dc66bc
Show file tree

Hide file tree

Showing 3 changed files with 97 additions and 10 deletions.
diff --git a/ui/src/backoffice/authors/containers/AuthorDetailPageContainer.tsx b/ui/src/backoffice/authors/containers/AuthorDetailPageContainer.tsx
@@ -306,16 +306,22 @@ const AuthorDetailPageContainer = ({
                     subTitle="SNow information"
                   >
                     {tickets && (
-                      <p className="mb0">
-                        See related ticket
-                        <a
-                          href={tickets?.first()?.get('ticket_url')}
-                          target="_blank"
-                        >
-                          {' '}
-                          #{tickets?.first()?.get('ticket_id')}
-                        </a>
-                      </p>
+                      <>
+                        <p className="mb0">See related tickets</p>
+                        <ul className="mb0">
+                          {tickets.map((ticket: any) => (
+                            <li className="mb0">
+                              <a
+                                key={ticket.get('ticket_id')}
+                                href={ticket.get('ticket_url')}
+                                target="_blank"
+                              >
+                                #{ticket.get('ticket_id')}
+                              </a>
+                            </li>
+                          ))}
+                        </ul>
+                      </>
                     )}
                   </ContentBox>
                   {isSuperUserLoggedIn && (

diff --git a/ui/src/backoffice/authors/containers/__tests__/AuthorDetailPageContainer.test.tsx b/ui/src/backoffice/authors/containers/__tests__/AuthorDetailPageContainer.test.tsx
@@ -40,6 +40,10 @@ describe('AuthorDetailPageContainer', (ids: any = []) => {
             ids: [{ schema: 'ORCID', value: '0000-0002-6357-9297' }, ...ids],
           },
           status: 'approval',
+          tickets: [
+            { ticket_id: 'ticket1', ticket_url: 'www.ticket1.com' },
+            { ticket_id: 'ticket2', ticket_url: 'www.ticket2.com' },
+          ],
         }),
       }),
     });
@@ -160,6 +164,21 @@ describe('AuthorDetailPageContainer', (ids: any = []) => {
 
     expect(screen.getByText('Loading ...')).toBeInTheDocument();
   });
+
+  it('should not show all the tickets', () => {
+    renderComponent();
+
+    expect(
+      screen.getByRole('link', {
+        name: '#ticket1',
+      })
+    ).toBeInTheDocument();
+    expect(
+      screen.getByRole('link', {
+        name: '#ticket2',
+      })
+    ).toBeInTheDocument();
+  });
 });
 
 describe('AuthorDetailPageContainer - Name Fields and control number', () => {

diff --git a/...office/authors/containers/__tests__/__snapshots__/AuthorDetailPageContainer.test.tsx.snap b/...office/authors/containers/__tests__/__snapshots__/AuthorDetailPageContainer.test.tsx.snap
@@ -577,6 +577,68 @@ exports[`AuthorDetailPageContainer should render the AuthorDetailPageContainer 1
                 </div>
               </div>
             </div>
+            <div
+              class="__ContentBox__ mb3"
+            >
+              <div
+                class="ant-card ant-card-bordered h-100"
+              >
+                <div
+                  class="ant-card-body"
+                >
+                  <div
+                    class="pa2 mb3"
+                  >
+                    <h3
+                      class="pb1"
+                    >
+                      SNow information
+                    </h3>
+                    <div>
+                      <p
+                        class="mb0"
+                      >
+                        See related tickets
+                      </p>
+                      <ul
+                        class="mb0"
+                      >
+                        <li
+                          class="mb0"
+                        >
+                          <a
+                            href="www.ticket1.com"
+                            target="_blank"
+                          >
+                            #ticket1
+                          </a>
+                        </li>
+                        <li
+                          class="mb0"
+                        >
+                          <a
+                            href="www.ticket2.com"
+                            target="_blank"
+                          >
+                            #ticket2
+                          </a>
+                        </li>
+                      </ul>
+                    </div>
+                  </div>
+                  <div
+                    class="ant-row ant-row-space-between actions ph2"
+                  >
+                    <div
+                      class="ant-col"
+                    />
+                    <div
+                      class="ant-col"
+                    />
+                  </div>
+                </div>
+              </div>
+            </div>
             <div
               class="__ContentBox__ mb3"
             >