-
Notifications
You must be signed in to change notification settings - Fork 32
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
[EDITOR]: Prefill contact informations with current user #955
[EDITOR]: Prefill contact informations with current user #955
Conversation
Affected libs:
|
📷 Screenshots are here! |
926a26a
to
2646249
Compare
2646249
to
eb067e6
Compare
de199a6
to
e65998c
Compare
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.
Thank you, great work!!
export const REGION_HAUT_DE_FRANCE_ORG_FIXTURE = (): Organization => ({ | ||
name: 'Région Hauts-de-France', | ||
description: 'A very sweet description about Région Hauts-de-France.', | ||
email: '[email protected]', | ||
website: new URL('https://region-haut-de-france.com/'), | ||
logoUrl: new URL('https://region-haut-de-france.com/logo.png'), | ||
recordCount: 4, | ||
}) |
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.
I think it would be beneficial to agree on a way to write our fixtures. The original approach was to use deepFreeze
to avoid mutating fixtures between tests; creating an object each time makes sense too but it'd be better not to mix these practices too much to avoid confusion
e65998c
to
6e317bc
Compare
Description
This PR makes every new record created prefilled with current connected user as contact for resource with role "point_of_contact".
Screenshots
Quality Assurance Checklist
breaking change
labelbackport <release branch>
label**This work is sponsored by **.