Bi-weekly summary of handbook changes

@team

Summary of merge requests for opencraft/documentation/public from 2025-03-18

Merge request: docs: remove/replace mentions of OCIM
Description: ## Description

This PR remove or replaces mentions of OCIM in our public handbook

Testing instructions

Please provide detailed step-by-step instructions for testing this change.

Other information

Private Ref : BB-9386

Author: Kaustav Banerjee

For more details visit: #603.


Merge request: fix: Accidental early return on zoom account loop.
Description: This pull request fixes a bug in the zoom migration code where only the first zoom account might be honored due to an early return.

JIRA tickets: Log in - OpenCraft

Testing instructions:

  1. make install_prereqs
  2. Copy all the CI variables locally in an .env file (not because .env is respected but because it isn’t tracked by source control) in this manner:
export GOOGLE_API_CREDENTIALS=$(
cat <<'END_HEREDOC'
{
  "creds_or_token": {"token": "token-here", "refresh_token": "refresh-token-here", "token_uri": "https://oauth2.googleapis.com/token", "client_id": "client-id-here", "client_secret": "secret-here", "scopes": ["https://www.googleapis.com/auth/drive.file", "https://www.googleapis.com/auth/drive.metadata"], "expiry": "expiry-date"},
  "scopes": ["https://www.googleapis.com/auth/drive.file", "https://www.googleapis.com/auth/drive.metadata"]
}
END_HEREDOC
)
  1. source .env to load the CI variables (make sure you set them all)
  2. Add print statements to verify the loop
  3. Comment out the email line, since the firewall will prevent this, apparently
  4. make migrate_zoom_meetings
  5. Once verified, delete the .env file

Preferably, if you have access to some, use test credentials instead of the real CI ones. Or trust me bro, it’s a one line obvious fix. How broken could it be?

Author: Fox Danger Piacenti

For more details visit: #602.


Merge request: docs: Clarify product team review role.
Description: This MR clarifies the role of product in the discovery process, and makes it clear that its involvement is not merely for adding a single line item concerning the design phase, but is also critical to the format and scope of the document.

JIRA tickets: Log in - OpenCraft

Discussions: Updates to Discovery Instructions - #11 by Fox

Merge deadline: “None” if there’s no rush, “ASAP” if it’s critical, or provide a specific date if there is one.

Testing instructions:

  1. Check the built version of the documentation to make sure the formatting is sensible.

Author: Fox Danger Piacenti

For more details visit: #601.


Merge request: [STAR-2983] Update info about non-billable accounts to reflect recent category changes.
Description: In the context of STAR-3920 we updated the categorization of non-billable cell and non-billable non-cell accounts; see this comment for details.

This PR updates the handbook to reflect the changes.

It also fixes some typos and removes some outdated information found along the way.

Testing

Tickets

Reviewers

  • @gabrieldamours

Author: Tim Krones

For more details visit: #600.


1 Like