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

feat: API History #594

Merged
merged 66 commits into from
Aug 12, 2024
Merged

feat: API History #594

merged 66 commits into from
Aug 12, 2024

Conversation

piotrpdev
Copy link
Collaborator

@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/css/custom.scss 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
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)
@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
Collaborator 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
4 checks passed
@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