Skip to content

Conversation

@srivastavarahull
Copy link
Contributor

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for tracking the template ID used to create a project by introducing a TemplateId property to the Project model and adding corresponding test methods to verify project retrieval functionality.

  • Added TemplateId property to the Project model to track which project template was used during creation
  • Introduced TestGetProjectAsync and TestGetProjectsAsync test methods to validate project retrieval operations

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
construction/accountadmin/source/Model/Project.gen.cs Added nullable TemplateId string property with XML documentation
construction/accountadmin/test/TestAccountAdminApi.cs Added project_id environment variable and two new test methods for project retrieval

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

public int? MemberCount { get; set; }

/// <summary>
///The ID of the project that was used as a template to create this project.
Copy link

Copilot AI Dec 5, 2025

Choose a reason for hiding this comment

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

Missing space after the triple slashes in the XML documentation summary. Should be '/// The ID' instead of '///The ID'.

Suggested change
///The ID of the project that was used as a template to create this project.
/// The ID of the project that was used as a template to create this project.

Copilot uses AI. Check for mistakes.
///The ID of the project that was used as a template to create this project.
/// </summary>
/// <value>
///The ID of the project that was used as a template to create this project.
Copy link

Copilot AI Dec 5, 2025

Choose a reason for hiding this comment

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

Missing space after the triple slashes in the XML documentation value tag. Should be '/// The ID' instead of '///The ID'.

Suggested change
///The ID of the project that was used as a template to create this project.
/// The ID of the project that was used as a template to create this project.

Copilot uses AI. Check for mistakes.
@srivastavarahull srivastavarahull changed the base branch from main to development December 5, 2025 07:04
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.

4 participants