feat: Screentime warning

[razvan-pro]

This change is 

[KamilPawel]

Reviewed 7 of 7 files at r1, all commit messages.
Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @razvan-pro)

---

game_frontend/public/handlebars_template.html line 162 at r1 (raw file):

    var appData = {
      urlForAimmoDashboard: '{% url_for_aimmo_dashboard %}',
      {% if request.session.screentime_warning_timeout %}screentimeWarningTimeout: \{{ request.session.screentime_warning_timeout }},{% endif %}

Is that \ needed?

---

game_frontend/src/components/ScreentimeWarning/index.js line 1 at r1 (raw file):

import React, { useEffect } from 'react'

Wouldn't it be better to also import useState as well to be consistent - { useEffect, useState }?

---

game_frontend/src/components/ScreentimeWarning/index.js line 46 at r1 (raw file):

const ScreentimeWarning = () => {
  const [open, setOpen] = React.useState(false)

So then in both of these instances we can just use useState

---

game_frontend/src/components/ScreentimeWarning/index.js line 100 at r1 (raw file):

            screen breaks to recharge those brain cells!
          </Typography>
          <Box css={{ mt: 2 }}>

Is the css attr the old MUI way of applying styles?

[razvan-pro]

Reviewable status: 5 of 7 files reviewed, 4 unresolved discussions (waiting on @KamilPawel)

---

game_frontend/public/handlebars_template.html line 162 at r1 (raw file):

Previously, KamilPawel wrote…

Is that \ needed?

Yes, that is to escape those brackets to be interpreted by handlebars template - we want to pass this to Django (the generated template won't include the )

---

game_frontend/src/components/ScreentimeWarning/index.js line 1 at r1 (raw file):

Previously, KamilPawel wrote…

Wouldn't it be better to also import useState as well to be consistent - { useEffect, useState }?

👍

---

game_frontend/src/components/ScreentimeWarning/index.js line 46 at r1 (raw file):

Previously, KamilPawel wrote…

So then in both of these instances we can just use useState

👍

---

game_frontend/src/components/ScreentimeWarning/index.js line 100 at r1 (raw file):

Previously, KamilPawel wrote…

Is the css attr the old MUI way of applying styles?

Yeah, it took me a while to find why sx didn't work 😆

[KamilPawel]

Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @razvan-pro)

[codecov]

Codecov Report

Merging #1686 (adb59e2) into master (d960837) will increase coverage by 0.17%.
The diff coverage is 95.83%.

@@            Coverage Diff             @@
##           master    #1686      +/-   ##
==========================================
+ Coverage   67.22%   67.39%   +0.17%     
==========================================
  Files         171      173       +2     
  Lines        3521     3549      +28     
  Branches      298      305       +7     
==========================================
+ Hits         2367     2392      +25     
- Misses       1122     1125       +3     
  Partials       32       32              

Impacted Files	Coverage Δ
game_frontend/src/img/brain.svg	50.00% <ø> (ø)
...frontend/src/components/ScreentimeWarning/index.js	95.83% <95.83%> (ø)