Skip to content

initial release of Power App Template using reusable components#126

Merged
Katerina-Chernevskaya merged 4 commits intopnp:mainfrom
vipulj03:Sharing-PowerAppSamples
Mar 23, 2025
Merged

initial release of Power App Template using reusable components#126
Katerina-Chernevskaya merged 4 commits intopnp:mainfrom
vipulj03:Sharing-PowerAppSamples

Conversation

@vipulj03
Copy link
Contributor

  • 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.)

This sample helps in creating reusable components in Power Apps and also helps in creating a base template for faster development.

  • 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 @vipulj03,

Thank you for your sample! Great work!
To proceed with merging the sample I would like to ask you to implement connection with the data source via environment variable. This will ensure that during the import process makers will be able to select their SharePoint site and list, so after import the application will be ready to use without manual editing. This is the best practice. You can find how to do this in the following article: https://learn.microsoft.com/en-us/power-apps/maker/data-platform/environmentvariables-data-source-canvas-apps.

If you will have any questions on this update - feel free to ping me.

Thank you!

Best regards,
Katerina Ch.

@Katerina-Chernevskaya Katerina-Chernevskaya added the Needs Attention Something needs to be fixed with the PR before merging label Mar 10, 2025
Implemented connection with the data source via environment variable.
@vipulj03
Copy link
Contributor Author

Implemented connection with the data source via environment variable.

@Katerina-Chernevskaya Katerina-Chernevskaya removed the Needs Attention Something needs to be fixed with the PR before merging label Mar 23, 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 @vipulj03 ,

Thank you for adding environment variables! Now I'm going to merge your sample. Great work!

For the future improvements I would suggest you to do not hard code values in the code. You can dynamically pull required information from the data source. For this particular sample, you can pull ScreenValue from the SharePoint list rather than harcoded values:
image

Thank you for sharing this sample!

Regards,
Katerina Ch.

@Katerina-Chernevskaya Katerina-Chernevskaya merged commit 35b9c88 into pnp:main Mar 23, 2025
1 check passed
MrNICEForBonusWinner pushed a commit to MrNICEForBonusWinner/LLM-Analysis that referenced this pull request Jul 21, 2025
initial release of Power App Template using reusable components
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