pull_requests: show blocking requirements to satisfy

And set comment thread status active when commenting
the PR, so that adding a comment from the app behaves like the website.

Close #63

Author: sstasi95

lib/src/screens/pull_request_detail/base_pull_request_detail.dart

@@ -13,6 +13,7 @@ import 'package:azure_devops/src/mixins/share_mixin.dart';
 import 'package:azure_devops/src/models/commit.dart' as c;
 import 'package:azure_devops/src/models/project.dart';
 import 'package:azure_devops/src/models/pull_request.dart';
+import 'package:azure_devops/src/models/pull_request_policies.dart';
 import 'package:azure_devops/src/models/pull_request_with_details.dart';
 import 'package:azure_devops/src/router/router.dart';
 import 'package:azure_devops/src/services/ads_service.dart';

lib/src/screens/pull_request_detail/components_pull_request_detail.dart

@@ -224,6 +224,16 @@ class _PullRequestOverview extends StatelessWidget {
                     groupedFiles: ctrl.groupedConflictingFiles,
                   ),
                 ),
+              ] else if (ctrl.mustSatisfyPolicies) ...[
+                Container(
+                  decoration: BoxDecoration(
+                    borderRadius: BorderRadius.circular(AppTheme.radius),
+                    border: Border.all(color: context.colorScheme.error, width: 2),
+                  ),
+                  padding: const EdgeInsets.all(12),
+                  margin: const EdgeInsets.symmetric(vertical: 12),
+                  child: _MissingPolicies(ctrl: ctrl),
+                ),
               ],
               Text(
                 'Title:',
@@ -375,6 +385,41 @@ class _PullRequestOverview extends StatelessWidget {
   }
 }
 
+class _MissingPolicies extends StatelessWidget {
+  const _MissingPolicies({required this.ctrl});
+
+  final _PullRequestDetailController ctrl;
+
+  @override
+  Widget build(BuildContext context) {
+    return Column(
+      crossAxisAlignment: CrossAxisAlignment.start,
+      children: [
+        Text(
+          'This pull request cannot be completed because some requirements are not satisfied:',
+        ),
+        const SizedBox(height: 8),
+        ...ctrl.missingPolicies.map(
+          (p) => Padding(
+            padding: const EdgeInsets.only(bottom: 4),
+            child: Row(
+              children: [
+                Icon(
+                  DevOpsIcons.failed,
+                  color: context.colorScheme.error,
+                  size: AppTheme.isTablet ? 24 : 16,
+                ),
+                const SizedBox(width: 8),
+                Text(p.configuration?.type?.displayName ?? 'Unknown policy'),
+              ],
+            ),
+          ),
+        ),
+      ],
+    );
+  }
+}
+
 class _PullRequestChangedFiles extends StatelessWidget {
   const _PullRequestChangedFiles({
     required this.visiblePage,

lib/src/screens/pull_request_detail/controller_pull_request_detail.dart

@@ -33,12 +33,14 @@ class _PullRequestDetailController with ShareMixin, AppLogger, PullRequestHelper
 
   static const _requireMergeStrategyId = 'fa4e907d-c16b-4a4c-9dfa-4916e5d171ab';
 
-  bool get mustSatisfyPolicies =>
+  List<Policy> get missingPolicies =>
       prDetail.value?.data?.policies
           // filter out the merge strategy policy to allow completing the PR
           .where((p) => p.configuration?.type?.id != _requireMergeStrategyId && p.status != 'approved')
-          .isNotEmpty ??
-      false;
+          .toList() ??
+      [];
+
+  bool get mustSatisfyPolicies => missingPolicies.isNotEmpty;
 
   bool get mustBeApproved => reviewers.where((p) => p.reviewer.isRequired && p.reviewer.vote < 5).isNotEmpty;
 

lib/src/services/azure_api_service.dart

@@ -2085,6 +2085,7 @@ class AzureApiServiceImpl with AppLogger implements AzureApiService {
       prPath,
       body: threadId == null
           ? {
+              'status': ThreadStatus.active.intValue,
               'comments': [commentBody],
               if (filePath != null)
                 'threadContext': {