chore(deps): update dependency eslint-plugin-react-hooks to v7

[renovate]

This PR contains the following updates:

Package	Change	Age	Confidence
eslint-plugin-react-hooks (source)	6.1.1 -> 7.0.1

---

Release Notes

facebook/react (eslint-plugin-react-hooks)

v7.0.1

Compare Source

• Disallowed passing inline useEffectEvent values as JSX props to guard against accidental propagation. (#​34820 by @​jf-eirinha)
• Switch to export = so eslint-plugin-react-hooks emits correct types for consumers in Node16 ESM projects. (#​34949 by @​karlhorky)
• Tightened the typing of configs.flat so the configs export is always defined. (#​34950 by @​poteto)
• Fix named import runtime errors. (#​34951, #​34953 by @​karlhorky)

v7.0.0

Compare Source

This release slims down presets to just 2 configurations (recommended and recommended-latest), and all compiler rules are enabled by default.

• Breaking: Removed recommended-latest-legacy and flat/recommended configs. The plugin now provides recommended (legacy and flat configs with all recommended rules), and recommended-latest (legacy and flat configs with all recommended rules plus new bleeding edge experimental compiler rules). (@​poteto in #​34757)

---

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR was generated by Mend Renovate. View the repository job log.

[dargmuesli]

Adapted the code a bit, @soerface pls verify

[dargmuesli]

Also submitted this change upstream:

• docs(readme): use ref instead of state for sign in try authts/react-oidc-context#1836

[soerface]

Adapted the code a bit, @soerface pls verify

Thanks! Read briefly the linked article https://react.dev/learn/you-might-not-need-an-effect. I'm really not familiar with that stuff, just wondering how you came to the conclusion that useRef is the proper solution, as it doesn't seem to be mentioned in the article.

I'm currently without a laptop and therefore unable to test the change myself. It will take a couple weeks until I can have a closer look. If you feel confident that this is the right solution you can go ahead, but since you already opened PR upstream I would recommend waiting for feedback from the devs there. I've also subscribed to your upstream issue & PR.

Thank you so much for maintaining this!

[dargmuesli]

useEffect should be used for changes triggering rerenders and hasTriedSignin appears to be just a flag that prevents starting the signin process more than once. hasTriedSignin does not influence the JSX rendering as far as I see, so useRef can be used instead I think.

[renovate]

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

[dargmuesli]

Merging this as the linked upstream PR was merged by the maintainers :)