Skip to content

Commit

Permalink
PR feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
jkaster committed Nov 3, 2021
1 parent 75e6619 commit 3320732
Show file tree
Hide file tree
Showing 8 changed files with 16 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -31,16 +31,18 @@ import type { IProjectProps } from '../../models'
import { ProjectView } from '../../scenes'

interface ProjectViewDialogProps {
/** if assigned, dialog appears. If undefined, dialog is hidden */
project?: IProjectProps
closer: () => void
/** Dialog closing event handler */
onClose: () => void
}

export const ProjectViewDialog: FC<ProjectViewDialogProps> = ({
project,
closer,
onClose,
}) => {
const closeDialog = () => {
closer()
onClose()
}

return (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,16 +30,18 @@ import type { IJudgingProps } from '../../../models'
import { JudgingView } from './JudgingView'

interface JudgingViewDialogProps {
/** If assigned, Dialog appears. If undefined, dialog closes */
judging?: IJudgingProps
closer: () => void
/** Dialog closing event handler */
onClose: () => void
}

export const JudgingViewDialog: FC<JudgingViewDialogProps> = ({
judging,
closer,
onClose,
}) => {
const closeDialog = () => {
closer()
onClose()
}

return (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ export const JudgingList: FC<JudgingListProps> = () => {
pages={totalPages}
onChange={updatePage}
/>
<JudgingViewDialog judging={currentJudging} closer={closeView} />
<JudgingViewDialog judging={currentJudging} onClose={closeView} />
</>
)
}
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ export const ProjectList: FC<ProjectListProps> = () => {
pages={totalPages}
onChange={(pageNumber) => dispatch(updateProjectsPageNum(pageNumber))}
/>
<ProjectViewDialog project={currentProject} closer={closeView} />
<ProjectViewDialog project={currentProject} onClose={closeView} />
</>
)
}
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ import { useSelector } from 'react-redux'
import type { IProjectProps } from '../../../models'
import { ExtMarkdown } from '../../../components'
import { getTechnologies } from '../../../data/hack_session/selectors'
import { getMembers, techDescriptions } from './utils'
import { getMembers, techDescriptions } from '../../utils'

interface ProjectViewProps {
project: IProjectProps
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,4 +26,4 @@

export * from './ProjectList'
export * from './ProjectView'
export * from './utils'
export * from '../../utils'
1 change: 1 addition & 0 deletions packages/hackathon/src/scenes/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -33,3 +33,4 @@ export * from './AdminScene'
export * from './NotFoundScene'
export * from './ProjectEditorScene'
export * from './ResourceScene'
export * from './utils'
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
*/

import type { ITechnologyProps } from '../../../models'
import type { ITechnologyProps } from '../models'

export const techDescriptions = (
ids: string[],
Expand Down

0 comments on commit 3320732

Please sign in to comment.