codefymx / abpzero-smartadmin Goto Github PK

Dentro de algunas vistas parciales no se están cargando los textos traducidos.

Mediante el uso de los smart-forms estamos complicando la compatibilidad del sitio con otros frameworks.

Cambiar a y procurar el uso de de formularios de bootstrap normales.

Hello,

First, thank you for sharing your code.

Somehow, I got an error when I sent the email confirmation email.

The exception is happen the below method in CinotamMailSenderDefault:
`
async Task SendViaSmtp(MailMessage message)
{
try
{
var useSmtp = bool.Parse((await SettingManager.GetSettingValueAsync("UseSmtp")));

            if (!useSmtp) return false;
            await _emailSender.SendAsync(message);
            return true;
        }
        catch (Exception e)
        {
            Debug.WriteLine("Exception :" + e.Message);
            return false;
        }
    }

`

and the exception message is

Exception :The SMTP server requires a secure connection or the client was not authenticated. The server response was: 5.5.1 Authentication Required. Learn more at

I checked the SSL in the admin mail setting page like below. My email address and password are correct.

SMTP host: smtp.gmail.com
SMTP port: 587
username: my gmail address
password: my gmail password
domain name: gmail.com
Use SSL: checked
User default credentials: checked

Do you know what makes the exception?

-(El tenancyName es único)
-Esto nos permitirá cambiar el TenantId sin necesidad de que algún usuario inicie sesión en la aplicación.
-Útil para aplicaciones que requieran mostrar información de su empresa de forma publica sin necesidad de registros forzosos.

Agregar la funcionalidad faltante a la página de administración de empresas

Hello Alan,

I found one more thing.

If a new user required the email confirmation and the new user login the system using the email address as username, the user forward to the EmailNotConfirmed page. If the user click "Re-send email" button, it throw the login failed exception like below:

Login failed!

Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.

Exception Details: Abp.UI.UserFriendlyException: Login failed!

Source Error:

Line 315: {
Line 316: var user = await _userManager.Users.FirstOrDefaultAsync(a => a.UserName == userName);
Line 317: if (user == null) throw CreateExceptionForFailedLoginAttempt(loginResultType, userName, tenancyName);
Line 318: var token = await _userManager.GenerateEmailConfirmationTokenAsync(user.Id);
Line 319: var confirmationUrl = Url.Action("EmailConfirmation", new

Source File: C:\Users...\AbpCinotamZero-SPA-only-master\AbpCinotamZero-SPA-only-master\Cinotam.AbpModuleZero.Web\Controllers\AccountController.cs Line: 317

The reason is it searchs the userName to find the user object refer to the blow code. You may need to handle both userName and email address to find the user object or just allow userName only in the login page.

var user = await _userManager.Users.FirstOrDefaultAsync(a => a.UserName == userName);
if (user == null) throw CreateExceptionForFailedLoginAttempt(loginResultType, userName, tenancyName);

Thanks.

Una forma de discriminar el tipo de input necesario para los campos de configuración en la página Configurations

Actualmente las llaves para los servicios servicios de Cloudinary, Twilio y SendGrid son obtenidas desde las variables de entorno de windows; por lo tanto sería bueno agregar la opción de cargarlas desde el archivo de configuración de la aplicación

Se deben remover los archivos js que no se estén utilizando.

-El módulo Cms no está configurado para trabajar con MultiTenancy; en un principio AbpCinotamZero no estaba siendo desarrollada para trabajar con MultiTenancy por lo que algunas entidades muy antiguas del módulo cms no tienen implementada la interfaz IMustHaveTenant.

-Las entidades básicas de Abp y los servicios del módulo zero no tiene ningún problema hasta el momento.

En estos momentos se están adaptando las entidades y los managers para que pueda ser usado con esta característica.

Usar MultiTenancy.Resolvers
http://www.aspnetboilerplate.com/Pages/Documents/Multi-Tenancy#determining-current-tenant

La forma actual en la que se suben los archivos no permite que se puedan hacer pruebas unitarias de forma más sencilla; (se están enviando objetos de tipo HttpPostedFileBase al app service correspondiente), creo que seria recomendable dejar ese tipo de objetos en manos de los controladores.

Los textos de los lenguajes no se están actualizando conforme al archivo XML.