Skip to content
New issue

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

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

Already on GitHub? # to your account

Remove "directLink" prop #488

Merged
merged 1 commit into from
Dec 9, 2023
Merged

Remove "directLink" prop #488

merged 1 commit into from
Dec 9, 2023

Conversation

schroda
Copy link
Collaborator

@schroda schroda commented Dec 9, 2023

Is not required anymore since "Link" of react-router-dom is a "" element now anyway. It also caused the following warning

"react-dom.development.js:86 Warning: React does not recognize the directLink prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase directlink instead. If you accidentally passed it from a parent component, remove it from the DOM element"

Is not required anymore since "Link" of react-router-dom is a "<a>" element now anyway.
It also caused the following warning

"react-dom.development.js:86  Warning: React does not recognize the `directLink` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `directlink` instead. If you accidentally passed it from a parent component, remove it from the DOM element"
@schroda schroda merged commit db5d3ff into Suwayomi:master Dec 9, 2023
@schroda schroda deleted the fix/unrecognized_direct_link_prop branch December 9, 2023 13:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant