Skip to content
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

feat: add company signature form #159

Merged
merged 2 commits into from
Feb 25, 2025
Merged

feat: add company signature form #159

merged 2 commits into from
Feb 25, 2025

Conversation

serikjensen
Copy link
Member

This adds a company signature form as seen in the figma here https://www.figma.com/design/6dxOSiONDiJoa9zY1wOwbs/Frontend-SDK-Partner-Design-File?node-id=5220-17986&m=dev

Proof of functionality

Signature form submitting correctly and firing an event with the sign form response

Screen.Recording.2025-02-20.at.4.13.55.PM.mov

Form updated to complete in the document list

Screenshot 2025-02-20 at 4 14 38 PM

Form validation behaving correctly

Screen.Recording.2025-02-20.at.4.15.22.PM.mov

Download functionality behaving as expected

Screen.Recording.2025-02-20.at.4.17.49.PM.mov

Event firing for back navigation

Screen.Recording.2025-02-20.at.4.18.52.PM.mov

Mobile view

Screenshot 2025-02-20 at 4 20 33 PM

@@ -1,7 +1,7 @@
import { useTranslation } from 'react-i18next'

export function Head() {
const { t } = useTranslation('Company.DocumentSigner')
const { t } = useTranslation('Company.DocumentList')
Copy link
Member Author

Choose a reason for hiding this comment

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

No idea what happened with these but there must have been some regression 😅 it has always been named DocumentList. This updates to fix

/**
* TODO[GWS-3365]: This is a temporary value here until we resolve the correct way forward with the IP address
*/
signed_by_ip_address: '127.0.0.1',
Copy link
Member Author

Choose a reason for hiding this comment

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

This is something we still need to revisit. i think the consensus was that we just need to have them add the ip address via their proxy. Updated the ticket to generalize from employee document signing to company document signing

Copy link
Contributor

Choose a reason for hiding this comment

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

This is probably not the solution but looks like there are a lot of tools out there that let you hit their API and they return your IP address. Maybe Gusto needs an endpoint like this? https://api.myip.com/

Copy link
Collaborator

Choose a reason for hiding this comment

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

@jeffredodd we're in the browser, so we don't have a way to enforce this in an ironclad way (anyone can tamper with the output)

potentially there could be a way forward with redesigning the server API to allow for some kind of signature/nonce but it's not reachable without some groupwork :)

@serikjensen serikjensen merged commit b2dac30 into main Feb 25, 2025
1 check passed
@serikjensen serikjensen deleted the feat/GWS-3682 branch February 25, 2025 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants