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

New PR #77

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

New PR #77

wants to merge 1 commit into from

Conversation

ggkoolman
Copy link

@ggkoolman ggkoolman commented Feb 21, 2024

Reason:
@ggkoolman - Thanks for your PR. Can you please create a pull request for the following repository?

https://github.com/pnp/powerplatform-samples

We are in the process of moving to that single repo instead of having to maintain different repositories for all the different products (Power Apps, Power Automate, Power Pages & Copilot Studio).

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?

Solution init.

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.

  • [X ] 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

Reason:
@ggkoolman - Thanks for your PR. Can you please create a pull request for the following repository?

https://github.com/pnp/powerplatform-samples

We are in the process of moving to that single repo instead of having to maintain different repositories for all the different products (Power Apps, Power Automate, Power Pages & Copilot Studio).
@ggkoolman
Copy link
Author

Hi @Laskewitz,

I moved the sample here as requested.

Kind regards,
Gabriel

@Katerina-Chernevskaya
Copy link
Contributor

Katerina-Chernevskaya commented Mar 10, 2024

Hi @ggkoolman,

Thank you for your contribution!
I tried to pack the source code into a solution and got the following error:
image

Could you please check that all source code files are in place and upload them? Please follow the steps described in the guide to prepare the source code files.

Thank you!

Regards,
Katerina Ch.

@Katerina-Chernevskaya Katerina-Chernevskaya added the Needs Attention Something needs to be fixed with the PR before merging label Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Attention Something needs to be fixed with the PR before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants