resolved all the bugs

Author: muskansoninmh

cmd/add/accessRestriction/accessRestriction.go

@@ -66,9 +66,9 @@ func NewaccessRestrictionCmd() *cobra.Command {
 
 func addAccessRestriction(opts *accessRestrictionObj) error {
 	if opts.BlacklistDomain != "" || opts.WhitelistDomain != "" {
-		addDomian( opts.WhitelistDomain, opts.BlacklistDomain)
+		return addDomian( opts.WhitelistDomain, opts.BlacklistDomain)
 	} else if opts.AllowedIP != "" || opts.DeniedIP != "" {
-		addIpOrIpRange(opts.AllowedIP, opts.DeniedIP)
+		return addIpOrIpRange(opts.AllowedIP, opts.DeniedIP)
 	} else {
 		fmt.Println("Must use one of the following flags:")
 		fmt.Println("--blacklist-domain/whitelist-domain: To add either blacklist or whitelist Domain/Email")

cmd/delete/accessRestriction/accessRestriction.go

@@ -80,24 +80,12 @@ func NewaccessRestrictionCmd() *cobra.Command {
 				if len(newDomains) == 0 {
 					resp.ListType = "none"
 				}
-				deleteDomain(resp.ListType, newDomains)
+				return deleteDomain(resp.ListType, newDomains)
 
 
 			} else if opts.AllowedIP != "" || opts.DeniedIP != "" {
 
-				if opts.AllowedIP != "" {
-					isValid, err := cmdutil.ValidateIPorIPRange(opts.AllowedIP) 
-					if err != nil && !isValid {
-						fmt.Println("Error :" + err.Error())
-						return nil
-					}
-				} else if opts.DeniedIP != "" {
-					isValid, err := cmdutil.ValidateIPorIPRange(opts.DeniedIP) 
-					if err != nil && !isValid {
-						fmt.Println("Error :" + err.Error())
-						return nil
-					}
-				}
+				
 
 				siteFeatures, err := api.GetSiteFeatures()
 				if err != nil {
@@ -150,7 +138,7 @@ func NewaccessRestrictionCmd() *cobra.Command {
 						break
 					}
 				}
-				deleteIP(ipRestrictionType,newIPList)
+				return deleteIP(ipRestrictionType,newIPList)
 
 			}else if *allDomain {
 				var AddEmail api.EmailWhiteBLackListSchema 

cmd/get/accessRestriction/accessRestriction.go

@@ -42,9 +42,9 @@ Allowed/Denied IP or IP Range
 				return nil
 			}
 			if *domain {
-				GetDomainEmailList()
+				return GetDomainEmailList()
 			} else if *ip  {
-				GetIPList()
+				return GetIPList()
 			} else {
 				fmt.Println("Must use one of the following flags:") 
 					fmt.Println("--domain: To get the list of domains")
@@ -64,10 +64,7 @@ Allowed/Denied IP or IP Range
 
 func GetDomainEmailList() error {
 	resp, err := api.GetEmailWhiteListBlackList()
-			if err != nil && err.Error() == "No records found"{
-				fmt.Println(err.Error())
-				return nil
-			}
+		
 			if err != nil {
 				return err
 			}

cmd/login/login.go

@@ -98,7 +98,7 @@ func doLogin() error {
 	if err != nil {
 		return err
 	}
-	
+	fmt.Println("Fetching Permissions....")
 	err = api.GetPermissionsfromAPI()
 	if err != nil {
 		return err

cmd/resetSecret/resetSecret.go

@@ -86,7 +86,7 @@ func reset() error {
 	if err != nil {
 		return err
 	}
-	fmt.Println("API Secret reset successfully")
+	fmt.Println("The API Secret has been successfully reset.")
 
 	return nil
 }

cmd/set/smtpConfiguration/smtpConfiguration.go

@@ -29,7 +29,7 @@ func NewsmtpConfigurationCmd() *cobra.Command {
 		Example: heredoc.Doc(`
 		# SMTP Provider's Names we can use in set commands
 		# Mailazy, AmazonSES-USEast, AmazonSES-USWest, AmazonSES-EU, Gmail, 
-		Mandrill, Rackspace-mailgun, SendGrid, Yahoo, Other
+		Mandrill, Rackspace-mailgun, SendGrid, Yahoo, CustomSMTPProviders
 		$ lr set smtp-configuration -p Mailazy
 		? Key: <Key>
 		? Secret: <Secret>