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.

Sign up for GitHub

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

Jump to bottom

PerformanceReviewComponent_Hackathon #34

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

PerformanceReviewComponent_Hackathon #34

wants to merge 2 commits into from

Conversation

Chintalakiran
Copy link

@Chintalakiran Chintalakiran commented Sep 28, 2023 •
edited by gomomohapi
Loading

Category
new Sample

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?

This is a new sample for Performance review app

Checklist

  • 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

Chintalakiran
Chintalakiran commented Sep 28, 2023
View reviewed changes
Copy link
Author

@Chintalakiran Chintalakiran left a comment

Choose a reason for hiding this comment

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

Updated Project with Co-pilot screenshots and Video

@Katerina-Chernevskaya
Copy link
Contributor

Hello @Chintalakiran ,

Thank you for your submission!
To move forward with the review and merging of your pull request, we kindly request that you update your PR in accordance with the Contribution Guidelines. Please ensure that you added the README file following the provided template, and added the source code as instructed.

Thank you!

Regards,
Katerina Chernevskaya

@aprildunnam aprildunnam added the Needs Attention Something needs to be fixed with the PR before merging label Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Katerina-Chernevskaya Katerina-Chernevskaya Awaiting requested review from Katerina-Chernevskaya

Assignees
No one assigned
Labels
Needs Attention Something needs to be fixed with the PR before merging
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Chintalakiran @Katerina-Chernevskaya @aprildunnam