search

OpenNTI / nti.web.profiles

Other
0 stars 0 forks source link

Add components #37

Closed MahmoudMousaHamad closed 3 years ago

MahmoudMousaHamad commented 3 years ago

This pull request includes a couple of stories for example's sake

jsg2021 commented 3 years ago

because that’s dependent on the closed over value in scope. When the setting of that state is finally settled it may (and often is) out dated.

On Tue, Jan 19, 2021 at 7:58 AM Mahmoud Mousa Hamad < notifications@github.com> wrote:

@MahmoudMousaHamad commented on this pull request.

In src/social/View.js https://github.com/NextThought/nti.web.profiles/pull/37#discussion_r560193206 :

  • function toggleVisible () {
  • setVisible(!visible);
  • }

Why not just const toggle = x => setVisible(x ?? !visible);?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NextThought/nti.web.profiles/pull/37#discussion_r560193206, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEM7LMJAUPGJMS3BUSLUZTS2WFYRANCNFSM4WDIA6DA .