Aircall Connector

[tmuralikrishnan]

Closes #1297

Code Changes

NEW SaaS Connector Aircall for access

Steps to Confirm

• Add/configure the Aircall connector in UI
• Verify Aircall config and icon UI looks good
• Add customer data using UI
• Add calls data using Aircall App(no api for create calls)
• Create an Access DSR with the new Aircall customer phone number
• Verify email sent by fides with customer data is correct
• Create an Delete DSR with the new Aircall customer phone number
• Verify the customer data is now deleted in Aircall

Pre-Merge Checklist

• All CI Pipelines Succeeded
• Update CHANGELOG.md

SaaS Connector Checklist

• Spec doc link
• Vault Creds
• Postman Collection
• SaaS Config And Dataset
• Access and Erasure Test Passing
• Icon

[cypress]

Passing run #1326 ↗︎

0	3	0	0	0
⚠️ You've recorded test results over your free plan limit. Upgrade your plan to view test results.

Details:

Merge c7854df into 177ad01...
Project: fides	Commit: 8d967722da ℹ️
Status: Passed	Duration: 00:53 💡
Started: Apr 12, 2023 11:56 PM	Ended: Apr 12, 2023 11:57 PM

_{This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.}

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: +0.02 🎉

Comparison is base (4a864b2) 87.39% compared to head (c7854df) 87.41%.

❗ Current head c7854df differs from pull request most recent head 724d2b6. Consider uploading reports for the commit 724d2b6 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2589      +/-   ##
==========================================
+ Coverage   87.39%   87.41%   +0.02%     
==========================================
  Files         306      306              
  Lines       17598    17540      -58     
  Branches     2270     2255      -15     
==========================================
- Hits        15379    15333      -46     
+ Misses       1801     1793       -8     
+ Partials      418      414       -4     

see 4 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[tmuralikrishnan]

[tmuralikrishnan]

[tmuralikrishnan]

[tmuralikrishnan]

In user privacy request creation page phone number is required and email is optional for aircall.

[galvana]

Overall this looks good but we need to update the authentication strategy to basic. Also be mindful of not making changes to code that is not related to this connector (like fides.toml and the privacy center config) and leaving commented code