nathangeerinck / laravel-newsletter Goto Github PK

Hello Nathan,

Thank you for creating this, very grateful as I was testing it with mysql db and works well. However, I was testing this on SQL Server, and I keep getting an unauthorized error when creating a new list.

Beforehand, there were some migration errors as well such as below (produced from php artisan migrate, first initial migration):

SQLSTATE[42000]: [Microsoft][ODBC Driver 13 for SQL Server][SQL Server]Introducing FOREIGN KEY constraint 'subscriptions_mailing_list_id_foreign' on table 'subscriptions' may ca  
  use cycles or multiple cascade paths. Specify ON DELETE NO ACTION or ON UPDATE NO ACTION, or modify other FOREIGN KEY constraints. (SQL: alter table "subscriptions" add constrai  
  nt "subscriptions_mailing_list_id_foreign" foreign key ("mailing_list_id") references "mailing_lists" ("id") on delete cascade)  

Any thoughts? I've basically php artisan serve in Windows 10 and connectionstring is to a local sql server developer edition, also laravel 5.7 version since I ran composer install. I'm still looking into the database migration files, if they need some altering or order change for sql server, but would like to hear your input about this issue.

Thank you.

Regards,
Brian

It's time to upgrade to Laravel 5.5.

When I do a php artisan migrate:refresh I get an error:

[Illuminate\Database\QueryException]
SQLSTATE[HY000]: General error: 1553 Cannot drop index 'campaigns_template_id_foreign': needed in a foreign key constraint (SQL: alter table `campaigns` drop `template_id`)

[PDOException]
SQLSTATE[HY000]: General error: 1553 Cannot drop index 'campaigns_template_id_foreign': needed in a foreign key constraint

How to fix this? I figured out how to create an URL to a Mailing-List myself like this (using Demo-Seed):
http://test.local/subscribe/1

But it throws Error 500 and laravel.log indicates this

production.ERROR: View [subscribe.list] not found. {"userId":1,"email":"[email protected]","exception":"[object] (InvalidArgumentException(code: 0): View [subscribe.list] not found. at /var/www/projects/laravel/laravel-newsletter/public/vendor/laravel/framework/src/Illuminate/View/FileViewFinder.php:137)

Looks like there is a missing view?

It would also be great being able to generate/open the link from withing backend and List-View to the frontend URL (as new tab).

When I create a public list, and returns me to the show page of that list I get a 500 error:

Next ErrorException: Route [subscriptions.subscribe] not defined.

In the show.blade.php I commented this and then it worked:

@if($list->public == true)
    <tr>
        <th>Public link</th>
        <td><input class="form-control" value="{{ route('subscriptions.subscribe', $list) }}"></td>
    </tr>
@endif

Code redirect in RedirectIfAuthenticated.php redirects to /home.

if (Auth::guard($guard)->check()) {
    return redirect('/home');
}

This route does not exist.

Now, the language select is available at the application settings. We need to move this setting to a users account settings.

Maybe you can build it so that you can change language for the application inside the settings menu.

If you can make everything translated in EN files I can translate to Dutch if you want.

The APP_REGISTER .env variable is nowhere to be found, your readme say's to go to /register but then I get a 404.

Looking in the source I saw that you have to add a APP_REGISTER to your env to enable registering, but this is not added in de .env.example