fix: Move call to GitHub integration tasks out from trigger_feature_state_change_webhooks

[novakzaballa]

Thanks for submitting a PR! Please check the boxes below:

• I have run pre-commit to check linting
• I have added information to docs/ if required so people know about the feature!
• I have filled in the "Changes" section below?
• I have filled in the "How did you test this code" section below?
• I have used a Conventional Commit title for this Pull Request

Changes

• Move call to GitHub integration tasks out from trigger_feature_state_change_webhooks_signal for compatibility with versioning v2

How did you test this code?

• Create a GitHub integration, add any repository, link a feature with a GH issue, and update the feature values, show the comment in your GH issue

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 10, 2024 0:26am
flagsmith-frontend-preview	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 10, 2024 0:26am
flagsmith-frontend-staging	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 10, 2024 0:26am

[sentry-io]

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: api/features/tasks.py

Function	Unhandled Issue
trigger_feature_state_change_webhooks	TypeError: 'GithubData' object is not subscriptable /api/v1/environments/{environment_api_key}/featurestates/{p... Event Count: 1
trigger_feature_state_change_webhooks	AttributeError: 'Project' object has no attribute 'organisationn_id' /api/v1/environments/{environment_api_key}... Event Count: 1

---

_{Did you find this useful? React with a 👍 or 👎}

[github-actions]

Uffizzi Preview deployment-51466 was deleted.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.98%. Comparing base (b975831) to head (eafb5f5).
Report is 11 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3905      +/-   ##
==========================================
+ Coverage   95.92%   95.98%   +0.05%     
==========================================
  Files        1134     1135       +1     
  Lines       35958    36141     +183     
==========================================
+ Hits        34493    34689     +196     
+ Misses       1465     1452      -13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[khvn26]

Left two comments. Also, the code needs test coverage.

[matthewelwell]

Err... what? Why not just:

Suggested change

	github_configuration = (
	Organisation.objects.prefetch_related("github_config")
	.get(id=self.environment.project.organisation_id)
	.github_config.first()
	)
	github_configuration = (
	GithubConfig.objects.get(organisation_id=self.environment.project.organisation_id)
	)

[novakzaballa]

Long story, but basically I had issues with the soft-delete model and the one-to-one relationship and other things, but that complex query is not needed. Removed