Skip to content

Commit

Permalink
Merge pull request #1161 from ORCID/fix-affiliation-manager-checkbox-…
Browse files Browse the repository at this point in the history
…validation

fix affiliation manager checkbox validation
  • Loading branch information
bobcaprice authored Apr 12, 2024
2 parents 85757d2 + 69108f9 commit f3d1a4a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions ui/src/app/member/member.route.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,18 +9,18 @@ import { MemberUpdateComponent } from './member-update.component'
import { MemberDetailComponent } from './member-detail.component'
import { MemberImportPopupComponent } from './member-import-dialog.component'

export const MemberResolver: ResolveFn<Member | null> = (
export const MemberResolver: ResolveFn<Member> = (
route: ActivatedRouteSnapshot,
state: RouterStateSnapshot,
memberService: MemberService = inject(MemberService)
): Observable<IMember | null> => {
): Observable<IMember> => {
if (route.paramMap.get('id')) {
return memberService.find(route.paramMap.get('id')!).pipe(
filter<IMember>((member: IMember) => !!member),
take(1)
)
} else {
return of(null)
return of(new Member())
}
}

Expand Down

0 comments on commit f3d1a4a

Please sign in to comment.