Skip to content

chore: add Senegal to list of map center - #300

Merged
ralvarez-dg merged 2 commits into
mainfrom
task/add-senegal-to-map-center
Sep 25, 2026
Merged

ralvarez-dg merged 2 commits into
mainfrom
task/add-senegal-to-map-center

Conversation

@timothy-mugo

@timothy-mugo timothy-mugo commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Description

Related WP PR: devgateway/data-viz-wordpress#158

Type of change

  • Bug fix (fix:)
  • New feature (feat:)
  • Breaking change (BREAKING CHANGE:)
  • Refactor / chore (refactor: / chore:)
  • Documentation update (docs:)

Affected package(s)

  • @devgateway/dvz-ui-react
  • @devgateway/wp-react-lib
  • example only (no changeset needed)

Checklist

  • PR title follows Conventional Commits format
  • A changeset has been added (pnpm changeset) for any change to a published package
  • pnpm build passes locally
  • pnpm --filter @devgateway/* typecheck passes
  • No hardcoded credentials, internal URLs, client names, or PII introduced
  • Any new dependency has an Apache-2.0-compatible license

Copilot AI lite review requested due to automatic review settings September 25, 2026 09:00
@pkg-pr-new

pkg-pr-new Bot commented Sep 25, 2026

Copy link
Copy Markdown

Open in StackBlitz

pnpm add https://pkg.pr.new/@devgateway/dvz-ui-react@d112c87
pnpm add https://pkg.pr.new/@devgateway/wp-react-lib@d112c87

commit: d112c87

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Add the required patch changeset before merging.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 Medium severity

Open (1)
What changed in this PR

Adds Senegal as a selectable map center in the embeddable map component.

Changes:

  • Adds Senegal’s SEN entry with coordinates and map scale.
File Summary
packages/​dvz-ui/​src/​embeddable/​map/​index.jsx Registers Senegal in the map-center options. A patch changeset is required for the published package.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread packages/dvz-ui/src/embeddable/map/index.jsx
Copilot AI review requested due to automatic review settings September 25, 2026 09:02
@pkg-pr-new

pkg-pr-new Bot commented Sep 25, 2026

Copy link
Copy Markdown

Open in StackBlitz

pnpm add https://pkg.pr.new/@devgateway/dvz-ui-react@0eb088a
pnpm add https://pkg.pr.new/@devgateway/wp-react-lib@0eb088a

commit: 0eb088a

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The implementation is complete; only a non-blocking wording nit remains.

Review effort: Lite
Findings: None

Resolved since last review (1)

@ralvarez-dg
ralvarez-dg merged commit 86fb009 into main Sep 25, 2026
3 checks passed
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.

3 participants