Page MenuHomePhabricator
Differential D8745

[lib] Update id constants
ClosedPublic
Actions

Authored by michal on Aug 7 2023, 3:49 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jan 19, 8:21 PM
Unknown Object (File)
Sun, Dec 29, 12:27 PM
Unknown Object (File)
Sun, Dec 29, 12:27 PM
Unknown Object (File)
Sun, Dec 29, 12:20 PM
Unknown Object (File)
Sun, Dec 29, 11:53 AM
Unknown Object (File)
Fri, Dec 27, 5:58 PM
Unknown Object (File)
Dec 6 2024, 4:01 PM
Unknown Object (File)
Nov 19 2024, 5:08 AM
View All 62 Files
Subscribers
ashoat

Details

Reviewers
tomek
bartek
Commits
rCOMM72a3200ceb2c: [lib] Update id constants
Summary

ENG-4595
Not all ids were converted

Test Plan
  • for bots.js -> added a console.log in keyserver and web code and checked that only web had a prefix
  • for comm-staff-community (I replaced 311733 with id of one of communities on my dev env for testing)
    • checked id: '311733', sent an image on web, confirmed that there is no "encryptionKey" in extra column in uploads
    • checked id: process.env['KEYSERVER'] ? '311733' : '256|311733', sent an image on web, confirmed that there is "encryptionKey"
  • Searched for id: ', id = ', '1, '2 etc. (case insensitive) and checked if there are any other constants
  • Checked files in facts/ for any id constants

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
facts/
2 lines
2 lines

Diff 29581

View Options

lib/facts/bots.js

Loading...
View Options

lib/facts/comm-staff-community.js

Loading...