Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

piotrpdev
Copy link
Contributor

@piotrpdev piotrpdev commented Jul 2, 2024

Note

This PR is part of the 2024 Electron GSoC [Project] [Proposal].

Use export GH_TOKEN=<token> before yarn start if you want to test the releases fetching and unzipping.

You might have to manually modify sidebars.js to make yarn start work.

Note

In development mode, clicking on one of the breaking changes in the table will open the correct page but not jump to the linked header. This is an issue with Docusaurus.

TODO

@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 July 2, 2024 17:08 Inactive
@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 July 2, 2024 17:21 Inactive
src/transformers/api-history.ts Outdated Show resolved Hide resolved
src/css/custom.scss Outdated Show resolved Hide resolved
src/transformers/api-history.ts Outdated Show resolved Hide resolved
src/components/ApiHistoryTable.tsx Outdated Show resolved Hide resolved
@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 July 4, 2024 19:51 Inactive
@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 July 4, 2024 20:06 Inactive
@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 July 29, 2024 00:18 Inactive
@piotrpdev piotrpdev mentioned this pull request Aug 3, 2024
18 tasks
@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 August 4, 2024 18:30 Inactive
@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 August 5, 2024 06:19 Inactive
Reported-by: David Sanders <dsanders11@ucsbalum.com>
@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 August 5, 2024 13:35 Inactive
@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 August 6, 2024 19:54 Inactive
src/components/ApiHistoryTable.tsx Outdated Show resolved Hide resolved
Copy link
Member

@dsanders11 dsanders11 left a comment

Choose a reason for hiding this comment

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

Added a couple of suggestions to simplify the code.

src/components/ApiHistoryTable.tsx Outdated Show resolved Hide resolved
src/components/ApiHistoryTable.tsx Outdated Show resolved Hide resolved
Co-authored-by: David Sanders <dsanders11@ucsbalum.com>
@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 August 11, 2024 23:14 Inactive
piotrpdev and others added 2 commits August 12, 2024 00:39
Co-authored-by: David Sanders <dsanders11@ucsbalum.com>
Based-on-patch-by: David Sanders <dsanders11@ucsbalum.com>
Reference: #594 (comment)
@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 August 11, 2024 23:40 Inactive
@piotrpdev piotrpdev requested a review from dsanders11 August 11, 2024 23:41
@dsanders11
Copy link
Member

I think this is ready. 🎉 @piotrpdev, can you go ahead and remove the docs/ changes? Then I'll go ahead and merge.

`git diff feat/api-history origin/main -- docs/latest/api/ | git apply`
@MarshallOfSound MarshallOfSound temporarily deployed to electronjsorg-new-pr-594 August 12, 2024 14:10 Inactive
@piotrpdev
Copy link
Contributor Author

@dsanders11
I think this is ready. 🎉 @piotrpdev, can you go ahead and remove the docs/ changes? Then I'll go ahead and merge.

Done, hopefully nothing breaks on merge 🤞

@dsanders11 dsanders11 merged commit 457bd50 into main Aug 12, 2024
@dsanders11 dsanders11 deleted the feat/api-history branch August 12, 2024 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:docs Related to rendering the documentation area:markdown area:prebuild Related to the prebuild scripts ✨ enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.