pagure

#2981 Fix the rtd hook
Merged by pingou. Opened by cverna.
cverna/pagure fix_rtd_hook  into  master

Download 2981.patch

Fixes #2308

We need to make sure to call db_object() on the hook so that we can access the rtd_hook attribute.

This PR also fixes some issues coming from the re architecture.

Let's create the session within the function so we can close it at the end

2 new commits added

  • Make it possible for the webapp to write in the repos
  • Fix the rtd hook by creating the database relation

Missing closing session?

I closed it at the end of the function https://pagure.io/pagure/pull-request/2981#_2,39

Seeing the next line, I doubt we reach the one you're referring to ;-)

:)

rebased onto 40d2e3e026ee6b016bb7f961c0ea38bd77e31db2

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.