Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NewProduct] Motorola Mobility #1453

Closed
wants to merge 7 commits into from
Closed

Conversation

moviuro
Copy link
Contributor

@moviuro moviuro commented Aug 2, 2022

Used the /we/ pages where available ("Rest of Europe"); /us/ otherwise; and if still that fails, anything goes.
No link for "moto g9", "moto e (2020)", "motorola one 5G UW ACE", "motorola one fusion"

products/motorola.md Outdated Show resolved Hide resolved
products/motorola.md Outdated Show resolved Hide resolved
@captn3m0
Copy link
Member

captn3m0 commented Aug 2, 2022

Could you add use a releaseLabel/releaseCycle split, similar to #1340. That will ensure that the data is available cleanly over the API.

releaseCycle should be the kebab-case version of the current name being used. (moto-g-pro, moto-g-power-2022 etc)

@usta usta changed the title New page: motorola.md [NewProduct] Motorola Mobility Aug 4, 2022
@usta usta added the new-product This PR adds a new product to the website. label Aug 4, 2022
@captn3m0
Copy link
Member

@moviuro What's the source for the release dates in this PR?

@moviuro
Copy link
Contributor Author

moviuro commented Oct 17, 2022

Release dates were pulled from individual pages about system updates: https://motorola-global-en-roe.custhelp.com/app/software-security-update/g_id/7112 -> "Select your Product" -> [Pick a product]

Motorola edge+ (2022) says:

Currently receiving regular security updates.

Device launched on February, 2022

Security updates will stop on February, 2025

@captn3m0
Copy link
Member

Thanks. I generated an updated version here: https://gist.github.com/captn3m0/069b0fd5879b8a1a549a7dfc45634173, that might be helpful in updating this PR.

I think we can add a activeSupport column, similar to how we're tracking it for Samsung Mobiles. If Motorola is promising a "Next Android", then we can mark it as actively supported.

@captn3m0
Copy link
Member

Okay, looks like we're still at 67 devices.

@captn3m0 captn3m0 self-requested a review October 17, 2022 16:39
@usta
Copy link
Member

usta commented Oct 18, 2022

@captn3m0 can we change this to something like this [adding as root component] ->
releaseLabel: {{"__RELEASE_CYCLE__" | replace:'-',' '}}
and remove all releaseLabel: tags which doesnt need modification ( just like (year) parts)
this will cleanup .md file's duplicated parts like

-   releaseCycle: "moto-g-pro"
    releaseLabel: "moto g pro"

Copy link
Member

@usta usta left a comment

Choose a reason for hiding this comment

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

^^

@captn3m0
Copy link
Member

Let me give that a try

Copy link
Member

@usta usta left a comment

Choose a reason for hiding this comment

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

link problem ( just tried one of them and for my luck it gived me 404 :D )

releaseDate: 2022-08-01
eol: 2025-08-01
support: true
link: https://www.motorola.in/smartphones-moto-g32/p
Copy link
Member

Choose a reason for hiding this comment

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

giving 404

Copy link
Member

Choose a reason for hiding this comment

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

moviuro and others added 7 commits December 12, 2022 15:24
Used the /we/ pages where available ("Rest of Europe"); /us/ otherwise; and if still that fails, anything goes.
No link for "moto g9", "moto e (2020)", "motorola one 5G UW ACE", "motorola one fusion"
Co-authored-by: Nemo <commits@captnemo.in>
Good luck to Motorola owners.

Co-authored-by: Nemo <commits@captnemo.in>
@marcwrobel
Copy link
Member

This PR was rather old. I just rebased from master.

@captn3m0
Copy link
Member

But overall looks like it will be a page difficult to maintain !

Agreed. This is also true of other device pages - Samsung, NVIDIA.

I tried to write some code to scrape dates from the Motorala website, but it didn't work out - too much complexity in how the pages are rendered.

@captn3m0 captn3m0 dismissed their stale review December 15, 2022 11:16

focusing on other PRs, will let others review this instead.

releaseLabel: "moto g play (2021)"
releaseDate: 2021-01-01
eol: 2023-01-01
link: https://www.motorola.com/we/smartphones-moto-g-play/p
Copy link
Member

Choose a reason for hiding this comment

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

releaseDate: 2022-08-01
eol: 2025-08-01
support: true
link: https://www.motorola.in/smartphones-moto-g32/p
Copy link
Member

Choose a reason for hiding this comment

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

releaseLabel: "moto g power (2021)"
releaseDate: 2021-01-01
eol: 2023-01-01
link: https://www.motorola.com/us/smartphones-moto-g-power/p
Copy link
Member

Choose a reason for hiding this comment

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

releaseLabel: "motorola razr 5g"
releaseDate: 2020-09-01
eol: 2022-09-01
link: https://www.motorola.com/we/smartphones-razr-5g
Copy link
Member

Choose a reason for hiding this comment

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

Link looks wrong, it returns a server error page.

releaseLabel: "moto g stylus"
releaseDate: 2020-04-01
eol: 2022-04-01
link: https://www.motorola.com/us/smartphones-moto-g-stylus/p
Copy link
Member

Choose a reason for hiding this comment

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

category: device
iconSlug: motorola
releasePolicyLink: https://motorola-global-en-roe.custhelp.com/app/software-upgrade/
activeSupportColumn: true
Copy link
Member

Choose a reason for hiding this comment

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

activeSupportColumn: true means support must be explicitly set for all release cycles.

This is required by :

assert isinstance(r["support"], (bool, date))

@marcwrobel
Copy link
Member

I converted my earlier comments to proper review comments.

@usta
Copy link
Member

usta commented Feb 10, 2023

@marcwrobel could you commit your requested changes on this PR whenever you have free time ?
This PR is waiting here for too long time. Lets we make your suggestions and merge it

@marcwrobel
Copy link
Member

@marcwrobel could you commit your requested changes on this PR whenever you have free time ?
This PR is waiting here for too long time. Lets we make your suggestions and merge it

Like @captn3m0 I am focusing on other PRs, feel free to review / apply my suggestions.

@marcwrobel
Copy link
Member

marcwrobel commented Oct 12, 2023

Closing this PR as nothing happened since more than 6 months (sources still available on https://github.com/moviuro/endoflife.date/tree/motorola).

@marcwrobel marcwrobel closed this Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-product This PR adds a new product to the website.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants