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

[8651] kiezradar api rules #5959

Merged
merged 1 commit into from
Jan 21, 2025
Merged

[8651] kiezradar api rules #5959

merged 1 commit into from
Jan 21, 2025

Conversation

goapunk
Copy link
Contributor

@goapunk goapunk commented Jan 21, 2025

Describe your changes
Follow up to #5932 which adds permissions, more tests, uses UserGeneratedContentModel for models with user content and resets the kiezradar migrations. This sadly means you have to scrap your local development database or try the following steps:

If you want to keep your database you could do the following steps:

# enter your sqlite3 db
sqlite3 db.sqlite3
# drop the kiezradar database tables
DROP TABLE meinberlin_kiezradar_searchprofile;DROP TABLE meinberlin_kiezradar_kiezradarquery;DROP TABLE meinberlin_kiezradar_projectstatus;DROP TABLE meinberlin_kiezradar_projecttype;DROP TABLE meinberlin_kiezradar_searchprofile_districts;DROP TABLE meinberlin_kiezradar_searchprofile_organisations;DROP TABLE meinberlin_kiezradar_searchprofile_project_types;DROP TABLE meinberlin_kiezradar_searchprofile_status;DROP TABLE meinberlin_kiezradar_searchprofile_topics;
# quit sqlite
.quit
# activate your venv if not done already
source venv/bin/activate   
# reset kiezradar migrations
python manage.py migrate --fake meinberlin_kiezradar zero
# run migrations again
python manage.py migrate

Tasks

  • PR name contains story or task reference
  • Steps to recreate and test the changes
  • Documentation (docs and inline)
  • Tests (including n+1 and django_assert_num_queries where applicable)
  • Changelog

@goapunk goapunk requested a review from m4ra January 21, 2025 16:34
@goapunk goapunk force-pushed the jd-2025-01-kiezradar-api-rules branch from 994ac46 to f745f4b Compare January 21, 2025 16:49
@goapunk goapunk merged commit e110236 into dev Jan 21, 2025
3 checks passed
@goapunk goapunk deleted the jd-2025-01-kiezradar-api-rules branch January 21, 2025 17:10
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