Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: Modal coverage area (IN-834) #422

Merged
merged 368 commits into from
Dec 20, 2023
Merged

feat: Modal coverage area (IN-834) #422

merged 368 commits into from
Dec 20, 2023

Conversation

Joaquinb2000
Copy link
Collaborator

@Joaquinb2000 Joaquinb2000 commented Apr 20, 2023 •
edited by jira bot
Loading

Pull Request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build-related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: IN-834

What is the new behavior?

  • Added CoverageArea component

Does this introduce a breaking change?

  • Yes
  • No

Other information

@vercel
Copy link

vercel bot commented Apr 20, 2023 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
inreach-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 20, 2023 10:20pm
inreach-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 20, 2023 10:20pm

@ghost
Copy link

ghost commented Apr 20, 2023 •
edited by ghost
Loading

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@vercel vercel bot temporarily deployed to Preview – inreach-app April 24, 2023 14:16 Inactive
@vercel vercel bot temporarily deployed to Preview – inreach-web April 24, 2023 14:17 Inactive
@JoeKarow JoeKarow added new-feature New features or options. automerge Enable Kodiak auto-merge labels Apr 24, 2023
@JoeKarow JoeKarow added the automerge Enable Kodiak auto-merge label Dec 20, 2023
@JoeKarow JoeKarow marked this pull request as ready for review December 20, 2023 22:13
@JoeKarow JoeKarow self-requested a review as a code owner December 20, 2023 22:13
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Dec 20, 2023

Quality Gate Failed Quality Gate failed

Failed conditions

D Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

idea Catch issues before they fail your Quality Gate with our IDE extension SonarLint SonarLint

@kodiakhq kodiakhq bot merged commit 490712b into dev Dec 20, 2023
19 of 21 checks passed
@kodiakhq kodiakhq bot deleted the IN-834-modal-coverage-area branch December 20, 2023 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JoeKarow JoeKarow JoeKarow approved these changes

Assignees
No one assigned
Labels
📦 api 📦 app automerge Enable Kodiak auto-merge 📦 db kodiak: merge.method = 'squash' Kodiak will squash merge this PR. new-feature New features or options. 📦 ui
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Joaquinb2000 @JoeKarow @InReach-svc