-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move submissions from /apply to user profile #638
Changes from 4 commits
bbb55b5
f0ad4cb
b11ee47
8525b8c
5863870
626cf15
9365a35
00110d9
5efd906
4affb1f
2f0c577
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -39,7 +39,6 @@ function AdminPage({ | |
} | ||
|
||
const router = useRouter() | ||
|
||
const tabs = [ | ||
{ | ||
name: 'bulk', | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would appreciate if you can still use the existing SettingsProps type, e.g. edit it to include submissions and type the props for the Settings page again. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm getting a lot of mismatch type errors when I do that– I believe because of getInitialProps. We can resolve during the meeting Friday |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,15 +5,18 @@ import FavoritesTab from 'components/Settings/FavoritesTab' | |
import MembershipRequestsTab from 'components/Settings/MembershipRequestsTab' | ||
import ProfileTab from 'components/Settings/ProfileTab' | ||
import HashTabView from 'components/TabView' | ||
import { NextPageContext } from 'next' | ||
import React, { ReactNode } from 'react' | ||
import { toast, TypeOptions } from 'react-toastify' | ||
import renderPage from 'renderPage' | ||
import styled from 'styled-components' | ||
import { UserInfo } from 'types' | ||
import { ApplicationSubmission, UserInfo } from 'types' | ||
import { OBJECT_NAME_TITLE, SHOW_MEMBERSHIP_REQUEST } from 'utils/branding' | ||
|
||
import SubmissionsPage from '~/components/Submissions' | ||
import { BG_GRADIENT, CLUBS_BLUE, WHITE } from '~/constants/colors' | ||
import { BORDER_RADIUS } from '~/constants/measurements' | ||
import { doBulkLookup } from '~/utils' | ||
|
||
const Notification = styled.span` | ||
border-radius: ${BORDER_RADIUS}; | ||
|
@@ -34,7 +37,7 @@ type SettingsProps = { | |
authenticated: boolean | null | ||
} | ||
|
||
const Settings = ({ userInfo, authenticated }: SettingsProps) => { | ||
const Settings = ({ userInfo, authenticated, submissions }) => { | ||
/** | ||
* Display the message to the user in the form of a toast. | ||
* @param The message to show to the user. | ||
|
@@ -67,6 +70,11 @@ const Settings = ({ userInfo, authenticated }: SettingsProps) => { | |
icon: 'bookmark', | ||
content: <FavoritesTab key="subscription" keyword="subscription" />, | ||
}, | ||
{ | ||
name: 'submissions', | ||
label: 'Submissions', | ||
content: () => <SubmissionsPage initialSubmissions={submissions} />, | ||
}, | ||
{ | ||
name: 'Requests', | ||
icon: 'user-check', | ||
|
@@ -97,4 +105,19 @@ const Settings = ({ userInfo, authenticated }: SettingsProps) => { | |
) | ||
} | ||
|
||
type BulkResp = { | ||
submissions: Array<ApplicationSubmission> | ||
} | ||
|
||
Settings.getInitialProps = async (ctx: NextPageContext) => { | ||
const data: BulkResp = (await doBulkLookup( | ||
['submissions', '/submissions/?format=json'], | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. cc @rm03 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Looks like the |
||
ctx, | ||
)) as BulkResp | ||
return { | ||
...data, | ||
fair: ctx.query.fair != null ? parseInt(ctx.query.fair as string) : null, | ||
} | ||
} | ||
|
||
export default renderPage(Settings) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious if there's any reason for this change, seems to pass lint regardless but the previous spacing was more consistent on first glance, IMO.