diff --git a/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityRole.cs b/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityRole.cs
index dc8cc0b067..72fe231174 100644
--- a/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityRole.cs
+++ b/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityRole.cs
@@ -1,6 +1,3 @@
-// Copyright (c) .NET Foundation. All rights reserved.
-// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
-
 using System;
 using System.Collections.Generic;
 using System.Collections.ObjectModel;
@@ -11,7 +8,6 @@ using Volo.ExtensionMethods.Collections.Generic;
 
 namespace Volo.Abp.Identity
 {
-    //TODO: Should set Id to a GUID (where? on repository?)
     //TODO: Properties should not be public!
 
     /// <summary>
@@ -47,11 +43,13 @@ namespace Volo.Abp.Identity
         /// <summary>
         /// Initializes a new instance of <see cref="IdentityRole"/>.
         /// </summary>
+        /// <param name="id">Role id.</param>
         /// <param name="name">The role name.</param>
-        public IdentityRole([NotNull] string name)
+        public IdentityRole(Guid id, [NotNull] string name)
         {
             Check.NotNull(name, nameof(name));
 
+            Id = id;
             Name = name;
         }
 
diff --git a/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUser.cs b/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUser.cs
index fe13fa2211..79858fdb7c 100644
--- a/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUser.cs
+++ b/src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUser.cs
@@ -10,10 +10,8 @@ using Volo.ExtensionMethods.Collections.Generic;
 
 namespace Volo.Abp.Identity
 {
-    //TODO: Should set Id to a GUID (where? on repository?)
     //TODO: Properties should not be public!
     //TODO: Add Name/Surname/FullName?
-    //TODO: Set max lenght for Id (set a default max lenght by default, like 64)
     
     public class IdentityUser : AggregateRoot, IHasConcurrencyStamp
     {