Skip to content
This repository was archived by the owner on Jul 21, 2025. It is now read-only.

Conversation

@akshay-online
Copy link
Collaborator

This pull request introduces several updates to the ADOGenerator project, focusing on improving error messages, updating repository handling, and refining templates for Azure DevOps configurations. The changes enhance clarity in error reporting, ensure proper repository management, and align templates with updated naming conventions and URLs.

Error Message Enhancements:

  • Updated error messages in methods like UpdateBoardColumn, UpdateCardFields, and UpdateCardStyles to include the boardType for better context when debugging issues. (src/ADOGenerator/Services/ProjectService.cs) [1] [2] [3]

Repository Management Improvements:

  • Added logic in ImportSourceCode to create a default repository if none exists and to set the isDefaultRepoTodetele flag accordingly. (src/ADOGenerator/Services/ProjectService.cs) [1] [2]

Template Updates:

  • Updated URLs in multiple JSON templates to reflect the new Azure DevOps URL structure (dev.azure.com instead of vstsdemodata.visualstudio.com). This affects templates for DL-AKS, DL-Ansible, DL-AzureFunctions, and DL-DeploymentGroups. [1] [2] [3] [4] [5] [6] [7] [8] [9]

Board and Workflow Refinements:

  • Renamed board columns and adjusted state mappings in BoardColumns.json to better align with Agile practices (e.g., "Active" renamed to "In Progress", "Closed" renamed to "Done"). (src/ADOGenerator/Templates/DL-DeploymentGroups/Teams/Deployment Group Team/BoardColumns.json) [1] [2]
  • Renamed "Stories" to "Backlog Items" in multiple files, including BoardRows.json, CardFields.json, and CardStyles.json, to reflect updated terminology. [1] [2] [3] [4]

Bug Behavior Update:

  • Changed bugsBehavior in TeamSetting.json to treat bugs as requirements instead of tasks, aligning with updated project management practices. (src/ADOGenerator/Templates/DL-DeploymentGroups/Teams/Deployment Group Team/TeamSetting.json)
  • Improved error messages in ProjectService.cs to include board type context.
  • Added a check for default repository deletion based on existing repositories.
  • Updated Azure DevOps URLs in multiple JSON files to the new format.
  • Cleaned up descriptions in TemplateSetting.json by removing HTML tags and rephrasing for clarity.
  • Refined error handling in Repository.cs to include timestamps and method context.
  • Enhanced response handling in BoardColumn.cs for better robustness.

- Improved error messages in ProjectService.cs to include board type context.
- Added a check for default repository deletion based on existing repositories.
- Updated Azure DevOps URLs in multiple JSON files to the new format.
- Cleaned up descriptions in TemplateSetting.json by removing HTML tags and rephrasing for clarity.
- Refined error handling in Repository.cs to include timestamps and method context.
- Enhanced response handling in BoardColumn.cs for better robustness.
@akshay-online akshay-online marked this pull request as ready for review May 7, 2025 06:15
- Simplified `HandleNewProjectCreation` by removing the `confirmedExtension` output parameter and directly assigning the result of `ValidateExtensions`.
- Updated `TryGetTemplateDetails` to streamline template folder checks and improve error messaging.
- Introduced a new `Parameters` dictionary in `ProjectService.cs` for better organization of user input values.
- Added `GetServiceEndPointCredentials` method to prompt for user credentials based on the selected template, with error handling.
- Updated placeholder replacement logic in `jsonCreateService` to utilize the new `Parameters` dictionary.
- Changed `versionSpec` in `Octopus.json` from `"3.*"` to `"4.*"` to reflect the updated task versioning.
@AdithyaGauravCanarys
Copy link
Contributor

This pull request introduces several updates to the ADOGenerator project, focusing on refactoring methods, improving error messages, adding new functionality for handling service endpoint credentials, and updating template URLs for consistency. Below is a categorized summary of the most important changes:

Refactoring and Code Simplification:

  • Removed the confirmedExtension parameter from the TryGetTemplateDetails method and updated its usage in HandleNewProjectCreation to simplify the method signature. [1] [2] [3] [4]

New Functionality:

  • Added a new private method GetServiceEndPointCredentials to handle user input for credentials based on the selected template, storing them in the model.Parameters dictionary for later use. [1] [2]

Error Message Improvements:

  • Enhanced error messages in several methods (UpdateBoardColumn, UpdateCardFields, UpdateCardStyles, ImportSourceCode) to include the board type for better debugging context. [1] [2] [3] [4]

Template URL Updates:

  • Updated template URLs in multiple JSON files to use a consistent base URL (https://dev.azure.com/vstsdemodata) instead of the older https://vstsdemodata.visualstudio.com format. [1] [2] [3] [4] [5] [6] [7]

Repository Handling:

  • Adjusted the ImportSourceCode method to create a repository if the default repository details are missing, ensuring smoother project setup.

These changes improve code maintainability, enhance user experience, and ensure consistency across templates.

@gloridelmorales gloridelmorales self-requested a review May 8, 2025 14:40
@gloridelmorales gloridelmorales merged commit 1001601 into main May 8, 2025
4 checks passed
@akshay-online akshay-online deleted the RepoUrlChanges branch June 18, 2025 13:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants