-
Notifications
You must be signed in to change notification settings - Fork 1
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
Migrate to the new standard schema #579
Conversation
Is this a test or a migration to the new schema @koechkevin ? |
This is a migration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏽
Only done a quick review, I'm sure the rest of the team will do a more thorough review.
I've also noticed two files: airtable.old.js
and ecostystem.old.js
in the lib/ecosystem
folder. Are they still being used or are they just on my local machine?
@koechkevin When updating the Contributors' Table, Telegram social media does not appear in the dropdown list. |
@kelvinkipruto then the column does not exist in airtable. Not a code problem |
Description
This PR updates charter africa to use new standard schema.
As per discussion and test of the originally implemented schema, the difference in time/size is negligible.
Fixes #559
Type of change
Please delete options that are not relevant.
Screenshots
Checklist: