Replace o2 with o2-new and remove o2-new#358
Merged
Merged
Conversation
brtbrt
force-pushed
the
mbertamini/1597-remove-o2-new
branch
from
July 9, 2026 23:37
a11c565 to
725d31c
Compare
brtbrt
requested review from
a team and
annefranca
and removed request for
a team and
annefranca
July 10, 2026 00:00
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Promotes the
o2-newicon set to become the canonicalo2brand and removeso2-newentirely. Figma is now synced to speak only abouto2, which points at the formero2-newsource.Changes
icons/o2now contains the formero2-newset (947 icons across filled/regular/light);icons/o2-newremoved.animated-icons/O2now equals the formerO2-new(error.json,success.json); the previousO2/oldandO2/resourcesassets andO2-newwere removed.brands.config.js): removedo2-newfromBRAND_ORDERand deleted its brand object; theo2brand now pointsdefaultFileIdat the new fileCjvgrHEIycSQ6exznxnFXT.figma-export.yml): dropped theo2-newchoice, repointedO2_FILE_URLto the new O2 Figma file, and removed theO2_NEW_FILE_URL/O2_NEW_URLvariables, theo2-newPR case, and the O2-new mention in the "all" PR body.package.json: removed the threeexport-o2-new-*scripts..gitignore: removed theo2-new-*.jsongenerated-config entries.README.mdandICON_TABLE.md; the O2 row now shows the promoted set with no separate O2-New entry.Required actions before merging
Unlike the Vivo-evolution PR, this change introduces no placeholders:
o2reuses the real, existing Figma file idCjvgrHEIycSQ6exznxnFXT. There is, however, one required action:O2_FIGMA_IDsecret in thefigma-importGitHub environment toCjvgrHEIycSQ6exznxnFXT(or remove it so the newdefaultFileIdis used). The export driver resolves the source asprocess.env[figmaEnv] || defaultFileId, so a stale secret pointing at the old O2 file (wHTqJ7KDhGKrNSNpmMb9nW) would silently override the new default and reverticons/o2to the old icons on the next export.