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

2024 07 04 ai and form fields #211

Merged
merged 34 commits into from
Jul 15, 2024
Merged

2024 07 04 ai and form fields #211

merged 34 commits into from
Jul 15, 2024

Conversation

Ches-ctrl
Copy link
Owner

No description provided.

@Ches-ctrl Ches-ctrl self-assigned this Jul 4, 2024
Copy link
Collaborator

@daniel-sussman daniel-sussman Jul 9, 2024

Choose a reason for hiding this comment

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

JB suggested I take a look and offer suggestions. Here are a few thoughts I had:

  • could we call this alternatively with job_data rather than with a url? That avoids calling the API a second time to get the application_criteria after first calling it to get the job_details, which is unnecessary if they're both part of the same json response (as with GH). Other ATSs will require a new API call to get the application_questions anyway.
  • I would put the logic that builds @fields in a separate method, so it can be easily swapped out for different ATSs
  • likewise, fetch_json should go in a separate method. It's sometimes going to be more complicated than with GH. With ashby, for example, this request is a GraphQL query via a POST request with specific info in the request body and headers.
  • just fyi, running FormFiller with GH jobs is a bit trickier if you're getting the questions via API, because mostly the API doesn't give you any useful locators other than the text of the question itself. I did some experimenting and verified that it's possible to get FormFiller to work using the text of the questions as locators, although it runs somewhat more slowly than if we get the element IDs via GetFormFields.

Copy link
Owner Author

Choose a reason for hiding this comment

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

This is helpful context thanks Dan - noted on the updates to the Faraday helper and also calling with job_data instead - ideally we do this as you note (we'll need to build the URL carefully for Greenhouse in this case as perhaps we'll sometimes want to pass the additional params and sometimes not? Keen to discuss / hear thoughts)

Copy link
Owner Author

Choose a reason for hiding this comment

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

Also a note on this - we can add further customisation into how Faraday makes the connection than what we're doing at the moment - think we might want to bring this in in future so just noting (e.g. custom user agent, proxy etc.)

@Ches-ctrl Ches-ctrl marked this pull request as ready for review July 15, 2024 11:58
@Ches-ctrl Ches-ctrl merged commit a0bb1d4 into main Jul 15, 2024
7 checks passed
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.

None yet

3 participants