Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

last Commit #1

Open
wants to merge 17 commits into
base: evaluate_me
Choose a base branch
from
Open

last Commit #1

wants to merge 17 commits into from

Conversation

SafaaMagdyAwad
Copy link
Owner

@SafaaMagdyAwad SafaaMagdyAwad commented Sep 5, 2024

Hello, how are you ,I hope you are having a good time. I did my best in this project as I...
*I speeded up email sending by using laravel Queues
*I also tried to avoid repetition as much as possible.

  • The front-end design has been maintained, although the code has become a little longer.
    *I tried to make the seeder more realistic in the data.
  • I also added my point of view on the login & registration codes
  • after login the user will be directed to admin dashboard instead of home
  • after registration user will be directed to login page.
  • user must be active and vertified to login
  • i made some changes in welcome.blade.php as well.
  • i used command to send newsletter emails to all active subscribed emails as there will be a set of emails well be send. I used laravel queues as well.

give me feedback about this code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant