pagure

#1015 Fix deleting a project properly from the DB
Merged by pingou. Opened by pingou.
fix_delete_hooks  into  master

Download 1015.patch
no initial comment

rebased

This should fix #883 and #884

rebased

As far as I am aware, unless you specified this name manually in the creation, the name is dependent on the database driver.

That is true, these are the PostgreSQL names and will break on other database, but there is no solution that would work for all databases.

I guess +1....
The model changes are good, but I find it really annoying that we break upgrade compatibility with anything non-postgres, even though we never mentioned that we only support postgres.
Preferably, you should probably add code to support all other engines, but I guess that's undoable..

rebased

3 new commits added

  • Drop the names of the foreign keys for sqlalchemy's naming conventions in the plugins
  • Drop the names of the foreign keys for sqlalchemy's naming conventions
  • Fix typos

ack.. still same annoyance, but looks good.

rebased

Thanks

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.