Skip to content

DBC22-6137: Added Esri map attribution - #1288

Merged
wmuldergov merged 2 commits into
mainfrom
feature/DBC22-6137
Apr 9, 2026
Merged

DBC22-6137: Added Esri map attribution#1288
wmuldergov merged 2 commits into
mainfrom
feature/DBC22-6137

Conversation

@minORC

@minORC minORC commented Apr 9, 2026

Copy link
Copy Markdown
Collaborator

📝 Min Ji Choi

🔗 JIRA Ticket


✅ Quality Assurance & Requirements

  • Requirements Met: I have confirmed that all acceptance criteria from the JIRA ticket are fulfilled.
  • Tested desktop in local or dev envs
  • Tested mobile in local or dev envs
  • Ran unit tests N/A
  • SonarCloud: I have verified that the SonarCloud analysis is clean/passing for this branch.

⚙️ Configuration & Environment

  • New Env Variables: Does this PR require new environment variables? No

    If yes, please list them here and ensure they are added to secret manager, the .env.example. and the Vault by an STA.

🧪 How to Test (if required)

  1. Navigate to Map page and Report a problem page
  2. Verify that the Esri map attribution is on both pages, as well as button functioning well to reveal the attribution on click, and closes as well on click.

🔍 Reviewer Checklist

  • Reviewed code for logic and cleanliness
  • Re-tested desktop/mobile in local or dev envs
  • Verified no new console warnings/errors
  • Confirmed that any new env variables are understood/documented

@minORC
minORC marked this pull request as ready for review April 9, 2026 21:19
@sonarqubecloud

sonarqubecloud Bot commented Apr 9, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
6.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@wmuldergov wmuldergov left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed it looks good on all areas we display a map.

@wmuldergov
wmuldergov merged commit abe3828 into main Apr 9, 2026
11 of 12 checks passed
@ray-oxd
ray-oxd deleted the feature/DBC22-6137 branch July 21, 2026 00:16
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