azure-docs icon indicating copy to clipboard operation
azure-docs copied to clipboard

Updated add-api.md to suit React Vite users as well

Open rvrama opened this issue 3 months ago • 8 comments

Made changes in 2 places for React Vite users to use dist folder instead of build. While building and while running with swa command locally. PLEASE IGNORE MY PREVIOUS COMMIT as this has complete change.

rvrama avatar Sep 28 '25 08:09 rvrama

@rvrama : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

prmerger-automator[bot] avatar Sep 28 '25 08:09 prmerger-automator[bot]

Learn Build status updates of commit b1e877a:

:warning: Validation status: warnings

File Status Preview URL Details
articles/static-web-apps/add-api.md :warning:Warning Details

articles/static-web-apps/add-api.md

  • Line 260, Column 1: [Warning: invalid-tab-group - See documentation] Tab group with different tab id set.
  • Line 307, Column 5: [Warning: invalid-tab-group - See documentation] Tab group with different tab id set.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@v1212

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

ttorble avatar Sep 29 '25 14:09 ttorble

@v1212

Can you review this old PR and determine whether it needs to be closed or merged?

@MicrosoftDocs/public-repo-pr-review-team

v-dirichards avatar Nov 04 '25 22:11 v-dirichards

Learn Build status updates of commit fc4d4ff:

:warning: Validation status: warnings

File Status Preview URL Details
articles/static-web-apps/add-api.md :warning:Warning Details

articles/static-web-apps/add-api.md

  • Line 260, Column 1: [Warning: invalid-tab-group - See documentation] Tab group with different tab id set.
  • Line 307, Column 5: [Warning: invalid-tab-group - See documentation] Tab group with different tab id set.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@rvrama @cjk7989 @v1212

The warnings must be fixed before this PR is ready to merge.

Once the warnings are clear, the commits must be moved to the private repository for automated checks and publishing. After you move the commits, close this PR (#please-close). If you need help moving the commits, contact the publicprs alias.

#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

v-dirichards avatar Nov 07 '25 15:11 v-dirichards

Learn Build status updates of commit fc4d4ff:

:warning: Validation status: warnings

File Status Preview URL Details
articles/static-web-apps/add-api.md :warning:Warning Details

articles/static-web-apps/add-api.md

  • Line 260, Column 1: [Warning: invalid-tab-group - See documentation] Tab group with different tab id set.
  • Line 307, Column 5: [Warning: invalid-tab-group - See documentation] Tab group with different tab id set.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

I sent email to the content owner today.

@MicrosoftDocs/public-repo-pr-review-team

ttorble avatar Dec 02 '25 09:12 ttorble