Skip to content

Conversation

@tahsinaislam
Copy link
Contributor

@tahsinaislam tahsinaislam commented Sep 25, 2025

Link to pivotal/JIRA issue

Is PM acceptance required? (delete one)

  • No - merge after code review approval

What was done?

  • Added airrecord gem to integrate airtable data
  • Using a test airtable to add orgs we have in demo/development to test
  • Adding integration now to 2026 Partners table to production, this will need to be updated each year

How to test?

  • Can be tested on console as such (I added Oregano Org to the test table):
Screenshot 2025-10-15 at 12 16 48 PM

Screenshots (for visual changes)

  • Before
  • After

@github-actions
Copy link

Heroku app: https://gyr-review-app-6066-a1b10df2649c.herokuapp.com/
View logs: heroku logs --app gyr-review-app-6066 (optionally add --tail)

@tahsinaislam tahsinaislam force-pushed the fyst-2779-add-airtable-integration branch from ea48325 to 7155809 Compare October 15, 2025 19:12
@tahsinaislam tahsinaislam force-pushed the fyst-2779-add-airtable-integration branch from 7155809 to 5a64176 Compare October 15, 2025 19:21
Copy link
Contributor

@jenny-heath jenny-heath left a comment

Choose a reason for hiding this comment

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

could you add tests for the model stuff?

@anisharamnani
Copy link
Contributor

once the staging credentials are added, i can approve! i just don’t want to approve beforehand and the comment gets lost.

Copy link
Contributor

@anisharamnani anisharamnani left a comment

Choose a reason for hiding this comment

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

@tahsinaislam tahsinaislam merged commit 7a6168e into main Oct 16, 2025
8 checks passed
@tahsinaislam tahsinaislam deleted the fyst-2779-add-airtable-integration branch October 16, 2025 22:17
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.

4 participants