Skip to content

fix: override target and rel links attribute in config #332

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
Merged
merged 1 commit into from
Aug 5, 2021
Merged

fix: override target and rel links attribute in config #332

merged 1 commit into from
Aug 5, 2021

Conversation

stafyniaksacha
Copy link
Contributor

The condition was overridden by the isExternal boolean

@stafyniaksacha
fix: override target and rel links attribute in config

Verified

This commit was signed with the committer’s verified signature.
dnzxy Deniz Cengiz
SSH Key Fingerprint: wFWesgvVbSNUQXg7VK/tRpvLym++hhlTzy2e9RQS7rA
Verified
Learn about vigilant mode
901f9b3
@posva posva merged commit 9d98dbb into vuejs:master Aug 5, 2021
YunYouJun pushed a commit to YunYouJun/vitepress that referenced this pull request Aug 30, 2021
@stafyniaksacha @YunYouJun
fix: override target and rel links attribute in config (vuejs#332)

Partially verified

This commit is signed with the committer’s verified signature.
dnzxy’s contribution has been verified via SSH key.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
667f15a
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@posva posva

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@stafyniaksacha @posva