Skip to content

Conversation

@ilbertt
Copy link
Member

@ilbertt ilbertt commented Aug 6, 2025

Description

Adds a job to the publish.yml workflow to zip the docs, push them to the icp-pages branch and trigger the workflow on the dfinity/icp-js-sdk-docs repo.

Depends on:

Additionally, this PR:

  • adds the versions dropdown on the docs sidebar as a plugin
  • moves the release-notes/v300 page to upgrading/v3

How Has This Been Tested?

A temporary action was created while this PR was open. That action has been removed before merging and its step copied to the publish.yml's publish-docs job.

Checklist:

  • My changes follow the guidelines in CONTRIBUTING.md.
  • The title of this PR complies with Conventional Commits.
  • I have edited the CHANGELOG accordingly.
  • I have made corresponding changes to the documentation.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 6, 2025

size-limit report 📦

Path Size
@dfinity/agent 50.37 KB (0%)
@dfinity/assets 55.38 KB (0%)
@dfinity/auth-client 21.15 KB (0%)
@dfinity/candid 13.22 KB (0%)
@icp-sdk/core 91 B (0%)
@dfinity/identity 20.59 KB (0%)
@dfinity/identity-secp256k1 33.88 KB (0%)
@dfinity/principal 4.44 KB (0%)
@dfinity/use-auth-client 61.27 KB (0%)

@ilbertt ilbertt changed the base branch from main to luca/upgrading-guide August 20, 2025 16:53
Base automatically changed from luca/upgrading-guide to main August 20, 2025 19:30
@ilbertt ilbertt mentioned this pull request Aug 20, 2025
4 tasks
@ilbertt ilbertt requested a review from nathanosdev August 20, 2025 21:08
@ilbertt ilbertt marked this pull request as ready for review August 20, 2025 21:10
@ilbertt ilbertt requested a review from nathanosdev August 21, 2025 09:56
@ilbertt ilbertt merged commit 767091a into main Aug 21, 2025
28 checks passed
@ilbertt ilbertt deleted the luca/SDK-2264-push-docs branch August 21, 2025 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants