Skip to content

VARify Bot Submission#141

Merged
Katerina-Chernevskaya merged 5 commits intopnp:mainfrom
BrianArcher94:VARify
Apr 14, 2025
Merged

VARify Bot Submission#141
Katerina-Chernevskaya merged 5 commits intopnp:mainfrom
BrianArcher94:VARify

Conversation

@BrianArcher94
Copy link
Contributor

By submitting this pull request, you agree to the contribution guidelines

If you aren't familiar with how to contribute to open-source repositories using GitHub, or if you find the instructions on this page confusing, sign up for one of our Sharing is Caring events. It's completely free, and we'll guide you through the process.

To submit a pull request with multiple authors, make sure that at least one commit is a co-authored commit by adding a Co-authored-by: trailer to the commit's message. E.g.: Co-authored-by: name <name@example.com>

Put an x in all the items that apply ([x], without spaces), make notes next to any that haven't been addressed.

  • Bug fix?
  • New sample?
  • Related issues: fixes #X, partially #Y, mentioned in #Z
  • Needs API permissions?
  • Has other prerequisites? (E.g. requires a list, document library, etc.)

What's in this Pull Request?

I would like to submit my VARify Bot project to the gallery. VARify Bot is a custom copilot agent that uses a third party API to fetch information related to football teams and fixtures.

Checklist

This checklist is mostly useful as a reminder of small things that can easily be forgotten – it is meant as a helpful tool rather than hoops to jump through.

Put an x in all the items that apply ([x], no spaces), make notes next to any that haven't been addressed.

  • My pull request affects only ONE sample.
  • I have updated the README file.
  • My README has at least one static high-resolution screenshot (i.e. not a GIF)
  • My README contains complete setup instructions, including pre-requisites and permissions required

@Katerina-Chernevskaya
Copy link
Contributor

Hi @BrianArcher94

Thank you for your submission! I was able to pack the solution, but when I tried to import it I got the missing dependencies error:
image
Could you please look at this and fix? Also please rename the main folder to keep powerplatform-samples without any changes.

Thank you!

Regards,
Katerina Ch.

@Katerina-Chernevskaya Katerina-Chernevskaya added the Needs Attention Something needs to be fixed with the PR before merging label Apr 14, 2025
@BrianArcher94
Copy link
Contributor Author

Hi @Katerina-Chernevskaya,

Can you expand the display name column so I can see the full name of the missing component please?

Thanks
B

@BrianArcher94
Copy link
Contributor Author

Hi @Katerina-Chernevskaya

I have updated my forked repository name and my solution & source code files.

I have tested it on my end and was able to import with no missing dependencies.

Thanks
B

@Katerina-Chernevskaya Katerina-Chernevskaya removed the Needs Attention Something needs to be fixed with the PR before merging label Apr 14, 2025
Copy link
Contributor

@Katerina-Chernevskaya Katerina-Chernevskaya left a comment

Choose a reason for hiding this comment

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

Hi @BrianArcher94 ,

Thank you! Now everything looks great! I'm going to merge the PR.

Thank you for your contribution! Please make sure that you registered in the recognition program https://aka.ms/community/recognition to get cool badges for your contribution! :)

Regards,
Katerina Ch.

@Katerina-Chernevskaya Katerina-Chernevskaya merged commit 65ab32b into pnp:main Apr 14, 2025
1 check passed
@BrianArcher94
Copy link
Contributor Author

Hi @Katerina-Chernevskaya Thanks for you help!

I have registered for the recognition program, how do I claim the badge?

Thanks
B

@BrianArcher94 BrianArcher94 deleted the VARify branch April 14, 2025 18:20
MrNICEForBonusWinner pushed a commit to MrNICEForBonusWinner/LLM-Analysis that referenced this pull request Jul 21, 2025
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.

2 participants