Skip to content

Commit d472f4a

Browse files
Merge pull request #50 from mozilla/settings-update
Settings builds vars before extracting env vars. Modifying approach to make see_also and jira_remote_link urls
2 parents 9277991 + 95c947f commit d472f4a

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

src/app/environment.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,13 +23,11 @@ class Settings(BaseSettings):
2323

2424
# Jira
2525
jira_base_url: str = "https://mozit-test.atlassian.net/"
26-
jira_issue_url: str = f"{jira_base_url}browse/%s"
2726
jira_username: str
2827
jira_api_key: str
2928

3029
# Bugzilla
3130
bugzilla_base_url: str = "https://bugzilla-dev.allizom.org"
32-
bugzilla_bug_url: str = f"{bugzilla_base_url}/show_bug.cgi?id=%s"
3331
bugzilla_api_key: str
3432

3533
# Logging

src/jbi/whiteboard_actions/default.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,11 +121,11 @@ def create_and_link_issue(self, payload):
121121
)
122122
return {"status": "duplicate", "jira_response": jira_response_delete}
123123
# else:
124-
jira_url = self.settings.jira_issue_url % jira_key_in_response
124+
jira_url = f"{self.settings.jira_base_url}browse/{jira_key_in_response}"
125125
update = self.bugzilla_client.build_update(see_also_add=jira_url)
126126
bugzilla_response = self.bugzilla_client.update_bugs([bug_obj.id], update)
127127

128-
bugzilla_url = self.settings.bugzilla_bug_url % bug_obj.id
128+
bugzilla_url = f"{self.settings.bugzilla_base_url}/show_bug.cgi?id={bug_obj.id}"
129129
jira_response = self.jira_client.create_or_update_issue_remote_links(
130130
issue_key=jira_key_in_response,
131131
link_url=bugzilla_url,

0 commit comments

Comments
 (0)