Merge branch 'main' into AN/fix-ui-entrypoint

freiheit-com · Aug 5, 2024 · 220286d · 220286d
2 parents c4b1de6 + 93a322c
commit 220286d
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 13 deletions.
diff --git a/services/frontend-service/src/ui/Pages/CommitInfo/CommitInfoPage.tsx b/services/frontend-service/src/ui/Pages/CommitInfo/CommitInfoPage.tsx
@@ -81,12 +81,6 @@ export const CommitInfoPage: React.FC = () => {
                 </div>
             );
         case CommitInfoState.READY:
-            return (
-                <CommitInfo
-                    commitInfo={commitInfo.response}
-                    triggerLoadMore={triggerLoadMore}
-                    pageNumber={pageNumber}
-                />
-            );
+            return <CommitInfo commitInfo={commitInfo.response} triggerLoadMore={triggerLoadMore} />;
     }
 };
diff --git a/services/frontend-service/src/ui/components/CommitInfo/CommitInfo.test.tsx b/services/frontend-service/src/ui/components/CommitInfo/CommitInfo.test.tsx
@@ -21,7 +21,7 @@ import { GetCommitInfoResponse, LockPreventedDeploymentEvent_LockType } from '..
 test('CommitInfo component does not render commit info when the response is undefined', () => {
     const { container } = render(
         <MemoryRouter>
-            <CommitInfo commitInfo={undefined} triggerLoadMore={null} pageNumber={0} />
+            <CommitInfo commitInfo={undefined} triggerLoadMore={null} />
         </MemoryRouter>
     );
     expect(container.textContent).toContain('Backend returned empty response');
@@ -260,7 +260,7 @@ test('CommitInfo component renders commit info when the response is valid', () =
         );
         const { container } = render(
             <MemoryRouter>
-                <CommitInfo commitInfo={testCase.commitInfo} triggerLoadMore={null} pageNumber={0} />
+                <CommitInfo commitInfo={testCase.commitInfo} triggerLoadMore={null} />
             </MemoryRouter>
         );
 
@@ -450,7 +450,7 @@ describe('CommitInfo component renders next and previous buttons correctly', ()
         it(testCase.name, () => {
             const { container } = render(
                 <MemoryRouter>
-                    <CommitInfo commitInfo={testCase.commitInfo} triggerLoadMore={null} pageNumber={0} />
+                    <CommitInfo commitInfo={testCase.commitInfo} triggerLoadMore={null} />
                 </MemoryRouter>
             );
 

diff --git a/services/frontend-service/src/ui/components/CommitInfo/CommitInfo.tsx b/services/frontend-service/src/ui/components/CommitInfo/CommitInfo.tsx
@@ -19,18 +19,16 @@ import { GetCommitInfoResponse, Event, LockPreventedDeploymentEvent_LockType } f
 type CommitInfoProps = {
     commitInfo: GetCommitInfoResponse | undefined;
     triggerLoadMore: { (): void } | null;
-    pageNumber: number;
 };
 
 export const CommitInfo: React.FC<CommitInfoProps> = (props) => {
     const commitInfo = props.commitInfo;
     const triggerLoadMore: () => void = props.triggerLoadMore !== null ? props.triggerLoadMore : (): void => {};
     const canLoadMore = props.commitInfo?.loadMore;
-    const pageNumber = props.pageNumber;
 
     const onClick = useCallback(() => {
         triggerLoadMore();
-    }, [pageNumber, triggerLoadMore]);
+    }, [triggerLoadMore]);
 
     if (commitInfo === undefined) {
         return (