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

Check for technical support contacts as part of new hub turn-up #3634

Merged
merged 2 commits into from
Jan 29, 2024

Conversation

yuvipanda
Copy link
Member

As part of #3048 (comment), we now have an airtable be the 'source of truth' for who is allowed to initiate support requests. This PR adds an item in the checklist for new hub turn up, making sure we validate that there is a technical contact for this hub.

The airtable in use may change soon as well as the terminology (as partnerships works on it #3048 (comment)). But it is important to have a verification step here, so we don't let whatever airtable it is be out of date.

Ref #3048 (comment)

As part of 2i2c-org#3048 (comment),
we now have an airtable be the 'source of truth' for who is
allowed to initiate support requests. This PR adds an item in
the checklist for new hub turn up, making sure we validate that
there *is* a technical contact for this hub.

The airtable in use may change soon as well as the terminology
(as partnerships works on it 2i2c-org#3048 (comment)).
But it is important to have a verification step here, so we don't
let whatever airtable it is be out of date.

Ref 2i2c-org#3048 (comment)
Copy link
Member

@sgibson91 sgibson91 left a comment

Choose a reason for hiding this comment

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

I might add this further up to the community information section? I feel like it's getting lost amongst technical detail (such as what image are they using, what authentication method, etc). And since it's Partnerships responsibility to act on updating the airtable (even if an engineer confirms that the information is missing), then it will probably benefit from being higher up in the list of tasks that need checking.

@yuvipanda
Copy link
Member Author

@sgibson91 great suggestion! I moved it up - what do you think?

Copy link
Member

@sgibson91 sgibson91 left a comment

Choose a reason for hiding this comment

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

Looks great!

@yuvipanda yuvipanda merged commit a760ca7 into 2i2c-org:master Jan 29, 2024
2 checks passed
@yuvipanda
Copy link
Member Author

image

Thanks for the review, folks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done 🎉
Development

Successfully merging this pull request may close these issues.

4 participants