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

add Ping to ICP Showcase #2326

Merged
merged 3 commits into from
Jan 16, 2024
Merged

add Ping to ICP Showcase #2326

merged 3 commits into from
Jan 16, 2024

Conversation

Saumay-Agrawal
Copy link
Contributor

@Saumay-Agrawal Saumay-Agrawal commented Jan 4, 2024

Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

@sa-github-api
Copy link

Dear @Saumay-Agrawal,

In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.

If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.

— The DFINITY Foundation

Copy link
Contributor

@jennifertrin jennifertrin left a comment

Choose a reason for hiding this comment

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

Can you please review the CLA steps above as well?

- remove empty fields
@jennifertrin
Copy link
Contributor

@Saumay-Agrawal Can you please resolve merge conflicts or allow us to edit the repo?

@Saumay-Agrawal
Copy link
Contributor Author

@jennifertrin I've resolved the merge conflicts.

@jennifertrin jennifertrin merged commit 1f32918 into dfinity:master Jan 16, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants