Skip to content

Commit e8224d3

Browse files
authored
Merge pull request #958 from CDLUC3/feature/950/JS-remove-page-scroll-from-template-create
Just removed scrollToTop on Template Create page
2 parents 2230e8f + a9be43e commit e8224d3

File tree

3 files changed

+14
-4
lines changed

3 files changed

+14
-4
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
### Fixed
88

99
### Removed
10+
- Removed `scrollToTop` from `Template Create` page [#950]
1011

1112
### Chore
1213
====================================================================================================================================

components/SelectExistingTemplate/index.tsx

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,6 @@ import {
3232
VersionedTemplateSearchResult,
3333
} from '@/generated/graphql';
3434

35-
// Hooks
36-
import { useScrollToTop } from '@/hooks/scrollToTop';
3735
// Other
3836
import { logECS, routePath } from '@/utils/index';
3937
import {
@@ -63,7 +61,6 @@ const TemplateSelectTemplatePage = ({ templateName }: { templateName: string })
6361
const topRef = useRef<HTMLDivElement>(null);
6462
const formatDate = useFormatDate();
6563
const router = useRouter();
66-
const { scrollToTop } = useScrollToTop();
6764
const toastState = useToast();
6865

6966
// State
@@ -165,7 +162,6 @@ const TemplateSelectTemplatePage = ({ templateName }: { templateName: string })
165162
const resetSearch = () => {
166163
setSearchTerm('');
167164
handleSearchInput('');
168-
scrollToTop(topRef);
169165
}
170166

171167

generated/graphql.tsx

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1869,6 +1869,12 @@ export type PlanFeedbackErrors = {
18691869
summaryText?: Maybe<Scalars['String']['output']>;
18701870
};
18711871

1872+
export enum PlanFeedbackStatusEnum {
1873+
Completed = 'COMPLETED',
1874+
None = 'NONE',
1875+
Requested = 'REQUESTED'
1876+
}
1877+
18721878
/** Funding associated with a plan */
18731879
export type PlanFunding = {
18741880
__typename?: 'PlanFunding';
@@ -2490,6 +2496,8 @@ export type Query = {
24902496
planFeedback?: Maybe<Array<Maybe<PlanFeedback>>>;
24912497
/** Get all of the comments associated with the round of admin feedback */
24922498
planFeedbackComments?: Maybe<Array<Maybe<PlanFeedbackComment>>>;
2499+
/** Get the feedback status for a plan (NONE, REQUESTED, COMPLETED) */
2500+
planFeedbackStatus?: Maybe<PlanFeedbackStatusEnum>;
24932501
/** Get all of the Funding information for the specific Plan */
24942502
planFundings?: Maybe<Array<Maybe<PlanFunding>>>;
24952503
/** Get all of the Users that are Members for the specific Plan */
@@ -2686,6 +2694,11 @@ export type QueryPlanFeedbackCommentsArgs = {
26862694
};
26872695

26882696

2697+
export type QueryPlanFeedbackStatusArgs = {
2698+
planId: Scalars['Int']['input'];
2699+
};
2700+
2701+
26892702
export type QueryPlanFundingsArgs = {
26902703
planId: Scalars['Int']['input'];
26912704
};

0 commit comments

Comments
 (0)