AlexsLemonade / refinebio-web

Refinebio Web
https://staging.web.refine.bio
BSD 3-Clause "New" or "Revised" License
1 stars 0 forks source link

222 - Integrate HubSpot API to the request data form for updating contact list #300

Closed nozomione closed 1 month ago

nozomione commented 9 months ago

Issue Number

Closing #222

Purpose/Implementation Notes

Changes includes:

Types of changes

Checklist

vercel[bot] commented 9 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
refinebio-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 7, 2024 9:26pm
nozomione commented 9 months ago

🗒️ Currently HubSpot API requests fail due to an unauthorized error 404 with the message This hapikey doesn't exist.

The dev error:

Screenshot 2023-12-06 at 9 19 35 PM

I've tested production and it also fails causing the CCDL-data-requesters contact list to remain unchanged (it seems that the date of the last contact added to the list was 10/14/2021).

The production error:

Screenshot 2023-12-06 at 8 05 52 PM Screenshot 2023-12-06 at 8 06 32 PM

Based on the error message, this may be resolved by updating our HubSpot API key 💡 Tagging @davidsmejia.

Update Note:

nozomione commented 9 months ago

Regarding 434e5e23903e9895012ba55f79a8c53ffb72ef1d, currently via FE, when a user submits a new data request using the same email address to submit the previous one, we will concatenate that new data to the existing data of the Dataset Request Details property (set as multi-line text), resulting in a long string.

Instead of string concatenation, alternately we can;

Let me know if this concatenation logic can be removed from the script!

The Dataset Request Details property history in a user contact page (it's a test data):

Screenshot 2023-12-08 at 8 14 54 PM

e.g.,) The concatenated property values on each request:

Screenshot 2023-12-11 at 3 01 58 PM