1620935 - Fix React warning in devtools/client/responsive/components/Browser.js `os: PropTypes.String`

Status: RESOLVED FIXED

(DevTools :: Responsive Design Mode, task, P3)

Description

[Julian Descottes [:jdescottes]]

What were you doing?

1. Build Firefox with the build option ac_add_options --enable-debug-js-modules
2. Run Firefox
3. Open RDM

What happened?

Several similar warnings about devices.phones[0].os are logged, for instance:

Warning: Failed prop type: App: prop type `devices.phones[0].os` is invalid; it must be a function, usually from the `prop-types` package, but received `undefined`.
in App (created by Connect(App))
in Connect(App)
in Provider

What should have happened?

No error should be logged.

Anything else we should know?

The fix should be easy

https://searchfox.org/mozilla-central/rev/557a0e222dd104c5d805ba344c45d6abc27d3db0/devtools/client/responsive/types.js#50

os: PropTypes.String,

should be changed to

os: PropTypes.string,

(uppercase String is not a valid PropType!)

This should be a good-first-bug for someone who wants to setup their development environment.
Out contributors documentation is at https://docs.firefox-dev.tools/

Comment 1

[michaelw.wang]

Hi,

I am new, however, I would like to work on this!

Thanks

Comment 2

[Julian Descottes [:jdescottes]]

Hi michaelw!

Thanks for offering to work on this, I'm assigning the bug to you.
Take a look at https://docs.firefox-dev.tools/ to get started, and don't hesitate to ask for help here or on slack/riot if needed.

Comment 3

[Julian Descottes [:jdescottes]]

Hi michaelw,

Do you need any help with the setup or with the bug?

Comment 4

[Julian Descottes [:jdescottes]]

No answer from the assignee, resetting.

Comment 5

[Alice :alisaur]

Hello! Could I work on this?
I'm new, and I just got my local build up and running :)

Comment 6

[Gabriel Luong [:gl] (ΦωΦ)]

(In reply to Alice :alisaur from comment #5)

Hello! Could I work on this?
I'm new, and I just got my local build up and running :)

Sure thing! I have assigned the issue to you.

Comment 7

[Alice :alisaur]

Are there any related tests that need to be updated? I would think no, but wanted to double check...

Comment 8

[Gabriel Luong [:gl] (ΦωΦ)]

(In reply to Alice :alisaur from comment #7)

Are there any related tests that need to be updated? I would think no, but wanted to double check...

No related tests need to be updated for this one.

Comment 9

[Alice :alisaur]

Attachment: Bug 1620935 - Fix device.os propTypes typo. r=gl

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fedfe4c9d40a
Fix device.os propTypes typo. r=gl

Comment 11

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/fedfe4c9d40a