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

Adding messages methods #3

Closed
wants to merge 5 commits into from

Conversation

HamzaHassanM
Copy link

@HamzaHassanM HamzaHassanM commented Mar 15, 2024

Changes Made in the Pull Request

1. Changes in README.md

  • Added:
    • Added instructions to run a command for configuring a file.

2. Changes in config/inbox.php

  • Added:
    • Added configuration options for the users' table and database search operator.

3. Changes in database/migrations/2024_02_17_000001_create_inbox_table.php

  • Modified:
    • Modified foreign key references to use the configured users table.

4. Changes in src/Facades/Inbox.php

  • Modified:
    • Updated the methods and their documentation.

5. Deleted src/Inbox.php

6. Changes in src/InboxServiceProvider.php

  • Added:
    • Added publishing of config file.
    • Registered a singleton for inbox service.

7. Added src/Model/Inbox.php

  • Added:
    • Added a model for inbox.

8. Deleted src/Services/Inbox.php

9. Added src/Services/InboxRepository.php

  • Added:
    • Added a repository for inbox.

10. Added src/Services/InboxService.php

  • Added:
    • Added a service for inbox operations.

11. Added src/Traits/ErrorExceptionTrait.php

  • Added:
    • Added a trait for handling error exceptions.

12. Changes in tests/Services/InboxTest.php

  • Modified:
    • Modified test case.

@mgamal92
Copy link
Owner

mgamal92 commented Mar 15, 2024

Thanks for your effort!.
This PR should be updated from 1.x branch.
@HamzaHassanM

@HamzaHassanM
Copy link
Author

Thanks for your effort!. This PR should be updated from 1.x branch. @HamzaHassanM

@mgamal92 Done

@HamzaHassanM HamzaHassanM deleted the add_methods branch October 17, 2024 12:35
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.

2 participants