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.

Sign up for GitHub

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

Jump to bottom

Add post_index signal #349

Merged
merged 3 commits into from Jul 25, 2021
Merged

Add post_index signal #349

merged 3 commits into from Jul 25, 2021

Conversation

saadmk11
Copy link
Contributor

@saadmk11 saadmk11 commented Jun 4, 2021

Closes #298

@safwanrahman
Copy link
Collaborator

Thanks @saadmk11 for the PR. It looks very nice with the docuemntation. I will take a closer look into it.

@saadmk11
Copy link
Contributor Author

saadmk11 commented Jun 6, 2021

@safwanrahman Thanks for reviewing the PR, I have updated the PR with the suggested changes.

safwanrahman
safwanrahman reviewed Jul 19, 2021
View reviewed changes
tests/test_signals.py Outdated Show resolved Hide resolved
@safwanrahman
Copy link
Collaborator

Looks Good to me! r+ 🤩🚀
@saadmk11 you rock! 🥳

@safwanrahman safwanrahman merged commit 6e15725 into django-es:master Jul 25, 2021
@saadmk11 saadmk11 deleted the add-post-index-signal branch July 25, 2021 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@safwanrahman safwanrahman Awaiting requested review from safwanrahman

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Signal post_index
2 participants
@saadmk11 @safwanrahman