-
Notifications
You must be signed in to change notification settings - Fork 15
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.
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
Add config to open external links in a new tab #326
Conversation
✅ Deploy Preview for ghg-demo ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Which commit or version are you updating VEDA UI to, @sandesh-sp, and why? I see the change in VEDA UI to use this setting has not been merged yet. |
@j08lue I might be unaware of the process. Should the changes on veda-ui be merged before I send a pull request here? |
Yes, we have not been requiring versioned releases on VEDA UI for instances to use, lately, but it should at least be merged to Btw, once we have turned VEDA UI into a library, hopefully later this year, you will have to use a packaged release. |
To avoid the merge conflicts for VEDA UI upgrade, perhaps make this PR just about the content changes and merge it only when VEDA UI has been updated (elsewhere). |
@j08lue that's the plan. for now, we just wanted to test the feature with this PR. We'll wait for the next veda-ui release to merge this PR. |
Ah, sorry I broke it then. Back to main for now. |
How to test this: See that the link to the IPCC-related website in the intro story
See that links in the footer still work as expected. |
Add config to open external links in a new browser tab
Closes #315