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

fix(bcd): Fix version range in production #7614

Merged
merged 3 commits into from
Nov 18, 2022

Conversation

danielhjacobs
Copy link
Contributor

@danielhjacobs danielhjacobs commented Nov 17, 2022

Summary

Fixes #6667 on prod

Problem

See #7589 (comment)

Solution

Put version_last (version before version_removed when version_removed is a string) in BCD info in document-extractor. Then show it in BCD version range.


Screenshots

No difference compared to #7589


How did you test this change?

yarn build

@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Nov 17, 2022
@danielhjacobs
Copy link
Contributor Author

Note: clashes with #7612

Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

LGTM, I just have some suggestions.

PS: Thank you! 🙏

build/document-extractor.ts Outdated Show resolved Hide resolved
build/document-extractor.ts Outdated Show resolved Hide resolved
@caugner caugner changed the title fix(build, bcd): Fix version range in production fix(bcd): Fix version range in production Nov 18, 2022
@caugner
Copy link
Contributor

caugner commented Nov 18, 2022

Note: clashes with #7612

Don't worry about that one, I'll have to redo that one anyways, as I mixed different aspects into a single commit, making it un-reviewable.

@caugner caugner marked this pull request as draft November 18, 2022 13:26
@danielhjacobs
Copy link
Contributor Author

Changed version_last type to VersionValue instead of string so it can be true when version_removed is true, which is necessary for the status = { isSupported: lastVersion ? "no" : "yes" }; check to work properly when version_removed is true.

@danielhjacobs danielhjacobs force-pushed the prod_earlier_version_removed branch from 9a9b4b4 to 8cd4627 Compare November 18, 2022 14:56
@danielhjacobs danielhjacobs marked this pull request as ready for review November 18, 2022 14:57
@danielhjacobs danielhjacobs force-pushed the prod_earlier_version_removed branch from 8cd4627 to 976aeb5 Compare November 18, 2022 15:00
@caugner caugner added the browser-compat issues related to the browser compatibility data tables (BCD) label Nov 18, 2022
@danielhjacobs danielhjacobs force-pushed the prod_earlier_version_removed branch from 976aeb5 to c65ab2f Compare November 18, 2022 15:18
@github-actions github-actions bot removed the browser-compat issues related to the browser compatibility data tables (BCD) label Nov 18, 2022
@danielhjacobs
Copy link
Contributor Author

Sorry, force-pushed to fix comment.

@caugner caugner merged commit 55044b5 into mdn:main Nov 18, 2022
@danielhjacobs danielhjacobs deleted the prod_earlier_version_removed branch November 18, 2022 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BCD tables include the version_removed in the version range
2 participants