Skip to content
This repository was archived by the owner on Nov 15, 2024. It is now read-only.

Conversation

razor-x
Copy link
Member

@razor-x razor-x commented Jun 13, 2023

No description provided.

@razor-x razor-x marked this pull request as ready for review June 13, 2023 16:38
@razor-x razor-x requested a review from codetheweb as a code owner June 13, 2023 16:38

export type CommonDeviceProperties = {
name: string
model: {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be optional?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It always returns something for the display name, sometimes it's like Unknown Model etc if we don't know.

@razor-x razor-x merged commit a4bb0c4 into main Jun 13, 2023
@razor-x razor-x deleted the device_model_prop branch June 13, 2023 17:08
seambot pushed a commit that referenced this pull request Jun 13, 2023
# [6.7.0](seamapi/javascript@v6.6.0...v6.7.0) (2023-06-13)

### Features

* Add model.display_name to CommonDeviceProperties ([#217](seamapi/javascript#217)) ([a4bb0c4](seamapi/javascript@a4bb0c4))
@seambot
Copy link
Contributor

seambot commented Jun 13, 2023

🎉 This PR is included in version 6.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants