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

feat: adds activity to admin management #1093

Merged
merged 1 commit into from
Jul 17, 2024
Merged

Conversation

yokwejuste
Copy link
Collaborator

@yokwejuste yokwejuste commented Jan 30, 2024

Closes #1094
image

@yokwejuste yokwejuste marked this pull request as ready for review January 30, 2024 11:55
@yokwejuste yokwejuste self-assigned this Jan 31, 2024
@NdibeRaymond
Copy link
Collaborator

we still need to rebase this right @yokwejuste ?

zubhub_backend/zubhub/activities/admin.py Outdated Show resolved Hide resolved
zubhub_backend/zubhub/activities/admin.py Outdated Show resolved Hide resolved
zubhub_backend/zubhub/activities/admin.py Outdated Show resolved Hide resolved
Copy link

github-actions bot commented Mar 1, 2024

pre-commit checks failed ❌❌❌
make sure you have pre-commit set up locally (see the README.md for instructions).
If you have set up pre-commit locally, you may need to undo this commit and
run "pre-commit" locally to see what the errors are.

Copy link

github-actions bot commented Mar 2, 2024

pre-commit checks failed ❌❌❌
make sure you have pre-commit set up locally (see the README.md for instructions).
If you have set up pre-commit locally, you may need to undo this commit and
run "pre-commit" locally to see what the errors are.

Copy link

pre-commit checks failed ❌❌❌
make sure you have pre-commit set up locally (see the README.md for instructions).
If you have set up pre-commit locally, you may need to undo this commit and
run "pre-commit" locally to see what the errors are.

3 similar comments
Copy link

pre-commit checks failed ❌❌❌
make sure you have pre-commit set up locally (see the README.md for instructions).
If you have set up pre-commit locally, you may need to undo this commit and
run "pre-commit" locally to see what the errors are.

Copy link

pre-commit checks failed ❌❌❌
make sure you have pre-commit set up locally (see the README.md for instructions).
If you have set up pre-commit locally, you may need to undo this commit and
run "pre-commit" locally to see what the errors are.

Copy link

pre-commit checks failed ❌❌❌
make sure you have pre-commit set up locally (see the README.md for instructions).
If you have set up pre-commit locally, you may need to undo this commit and
run "pre-commit" locally to see what the errors are.

* minor modifications to the admin model
* add admin models management to admin.py
* implement effective signals to handle
  admin models instances, addition,
  modification and deletion with proper
  cleanup

Issue: #1094
Signed-off-by: Ndibe Raymond Olisaemeka <rolisaemeka-ctr@wikimedia.org>
Copy link

All pre-commit checks passed ✅✅✅

@NdibeRaymond NdibeRaymond changed the base branch from master to staging July 17, 2024 02:50
@NdibeRaymond NdibeRaymond merged commit c3f4f34 into staging Jul 17, 2024
5 checks passed
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.

Adding activity management to the Django Admin
2 participants