Skip to content

PSP-11355: Visually associate primary contact and owning organization.#5331

Merged
stairaku merged 4 commits into
bcgov:devfrom
stairaku:PSP-11355
Jun 11, 2026
Merged

PSP-11355: Visually associate primary contact and owning organization.#5331
stairaku merged 4 commits into
bcgov:devfrom
stairaku:PSP-11355

Conversation

@stairaku

@stairaku stairaku commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@stairaku stairaku changed the title PSP-11355 PSP-11355: Visually associate primary contact and owning organization. Jun 9, 2026
@stairaku stairaku self-assigned this Jun 9, 2026
@stairaku stairaku added the enhancement New feature or request label Jun 9, 2026
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/5331

@eddherrera eddherrera self-requested a review June 10, 2026 22:54
teamMemberUrl,
primaryContactName,
primaryContactUrl,
showPrimaryContact = false,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are these default values needed? I haven't seen many of default values for props.

</SectionField>

{showPrimaryContact && (
<SectionField

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: please consider wrapping this with this a 'stiled.div' to apply font-style: Italic

tooltip?: string;
}

export const PrimaryContactSelectorView = ({

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rename to PrimaryContactSelectorDetails

@github-actions

Copy link
Copy Markdown
Contributor

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/5331

@stairaku stairaku marked this pull request as ready for review June 11, 2026 21:43
@sonarqubecloud

Copy link
Copy Markdown

@stairaku stairaku enabled auto-merge June 11, 2026 21:50
@stairaku stairaku added this pull request to the merge queue Jun 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/5331

Merged via the queue into bcgov:dev with commit 0f03336 Jun 11, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants