Add generate a map button to inspect external data source page #161
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…t in InspectExternalDataSource
…of display options back to the frontend pre-loaded
…ource inspect page
…report already exists for this data source and returns link to that report in error
commonknowledge-bot
deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-database PR #161
Render
Active
commonknowledge-bot
temporarily deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-backend PR #161
Render
Destroyed
commonknowledge-bot
temporarily deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-backend PR #161
Render
Destroyed
commonknowledge-bot
temporarily deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-frontend PR #161
Render
Destroyed
commonknowledge-bot
deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-worker PR #161
Render
Active
joaquimds
approved these changes
Dec 16, 2024
commonknowledge-bot
temporarily deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-backend PR #161
Render
Destroyed
commonknowledge-bot
deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-worker PR #161
Render
Active
commonknowledge-bot
temporarily deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-backend PR #161
Render
Destroyed
commonknowledge-bot
temporarily deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-worker PR #161
Render
Destroyed
commonknowledge-bot
had a problem deploying
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-worker PR #161
Render
Failure
commonknowledge-bot
temporarily deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-frontend PR #161
Render
Destroyed
commonknowledge-bot
temporarily deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-frontend PR #161
Render
Destroyed
commonknowledge-bot
temporarily deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-worker PR #161
Render
Destroyed
commonknowledge-bot
temporarily deployed
to
feat/MAP-354-add-view-this-on-report-button - meep-intelligence-hub-backend PR #161
Render
Destroyed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR adds the following:
A generate map for this data source button on the inspect external data source page that loads the data source onto a map and redirects the user to it
A check so that if a map with this data source already exists we provide a link to that map
Motivation and Context
Addresses issue MAP-354
How Can It Be Tested?
Download the branch and run locally
Import an external data source
Click on the Generate map for this data source button and observe that it redirects to a map with this data source loaded
Try to click on the button a second time and observe that the toast message offers a link to the existing map
How Will This Be Deployed?
Normal deployment process