Page MenuHomePhabricator
Differential D9383

[native] introduce avatar constants
ClosedPublic
Actions

Authored by ginsu on Oct 5 2023, 6:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 4, 5:58 PM
Unknown Object (File)
Thu, Apr 3, 10:25 PM
Unknown Object (File)
Wed, Apr 2, 8:00 PM
Unknown Object (File)
Wed, Apr 2, 7:02 PM
Unknown Object (File)
Mar 8 2025, 9:50 AM
Unknown Object (File)
Mar 4 2025, 12:10 PM
Unknown Object (File)
Mar 4 2025, 12:10 PM
Unknown Object (File)
Mar 4 2025, 12:10 PM
View All 90 Files
Subscribers
ashoat
atul
tomek
will

Details

Reviewers
atul
inka
Commits
rCOMM55b9a6cb5077: [native] introduce avatar constants
Summary

This diff address @atul's brittle code concerns in D9376

Depends on D9382

Test Plan

confirmed that there were no regressions with the sizing of avatars

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu created this revision.Oct 5 2023, 6:07 PM
Herald added subscribers: will, tomek, ashoat. · View Herald TranscriptOct 5 2023, 6:07 PM
ginsu added reviewers: atul, inka.Oct 5 2023, 6:08 PM
ginsu edited the summary of this revision. (Show Details)
ginsu added a parent revision: D9382: [native] introduce UserProfileMenuButton.
ginsu mentioned this in D9376: [native] introduce UserProfileAvatarModal.
Harbormaster completed remote builds in B23028: Diff 31708.Oct 5 2023, 6:26 PM
ginsu requested review of this revision.Oct 5 2023, 6:26 PM
ginsu added a child revision: D9261: [native] introduce useNavigateToUserProfileBottomSheet hook.Oct 6 2023, 3:43 AM
atul accepted this revision.Oct 6 2023, 7:58 AM

Thanks for addressing

This revision is now accepted and ready to land.Oct 6 2023, 7:58 AM
ginsu updated this revision to Diff 31831.Oct 9 2023, 1:43 PM

rebase before landing

Harbormaster completed remote builds in B23124: Diff 31831.Oct 9 2023, 2:01 PM
Closed by commit rCOMM55b9a6cb5077: [native] introduce avatar constants (authored by ginsu). · Explain WhyOct 9 2023, 2:18 PM
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMM55b9a6cb5077: [native] introduce avatar constants.

Revision Contents
Changeset List

PathSize
native/
avatars/
13 lines
44 lines
user-profile/
5 lines

Diff 31844

View Options

native/avatars/avatar-constants.js

Show First 20 Lines • Show All 55 Lines • ▼ Show 20 Lines networks:
- services-net - services-net
build: build:
dockerfile: services/identity/Dockerfile dockerfile: services/identity/Dockerfile
context: ../ context: ../
image: commapp/identity-server:0.1 image: commapp/identity-server:0.1
container_name: identity-server container_name: identity-server
ports: ports:
- "${COMM_SERVICES_PORT_IDENTITY}:50051" - "${COMM_SERVICES_PORT_IDENTITY}:50051"
# localstack
localstack:
image: localstack/localstack
container_name: localstack
hostname: localstack
ports:
- "4566:4566"
environment:
- SERVICES=s3,dynamodb
- DATA_DIR=${TMPDIR}/localstack/data
- HOSTNAME_EXTERNAL=localstack
volumes:
jimUnsubmitted
Not Done
Inline Actions

Why bind mount a temp directory instead of using a data volume? It's just gonna create issues with permissions. Also ${TMPDIR} is not used to refer to a path in both the host and the container which is a bad idea.

jim: Why bind mount a temp directory instead of using a data volume? It's just gonna create issues…
- ${TMPDIR}:/tmp/localstack
networks:
- services-net
View Options

native/avatars/avatar.react.js

Show First 20 Lines • Show All 55 Lines • ▼ Show 20 Lines networks:
- services-net - services-net
build: build:
dockerfile: services/identity/Dockerfile dockerfile: services/identity/Dockerfile
context: ../ context: ../
image: commapp/identity-server:0.1 image: commapp/identity-server:0.1
container_name: identity-server container_name: identity-server
ports: ports:
- "${COMM_SERVICES_PORT_IDENTITY}:50051" - "${COMM_SERVICES_PORT_IDENTITY}:50051"
# localstack
localstack:
image: localstack/localstack
container_name: localstack
hostname: localstack
ports:
- "4566:4566"
environment:
- SERVICES=s3,dynamodb
- DATA_DIR=${TMPDIR}/localstack/data
- HOSTNAME_EXTERNAL=localstack
volumes:
jimUnsubmitted
Not Done
Inline Actions

Why bind mount a temp directory instead of using a data volume? It's just gonna create issues with permissions. Also ${TMPDIR} is not used to refer to a path in both the host and the container which is a bad idea.

jim: Why bind mount a temp directory instead of using a data volume? It's just gonna create issues…
- ${TMPDIR}:/tmp/localstack
networks:
- services-net
View Options

native/user-profile/user-profile-avatar-modal.react.js

Show First 20 Lines • Show All 55 Lines • ▼ Show 20 Lines networks:
- services-net - services-net
build: build:
dockerfile: services/identity/Dockerfile dockerfile: services/identity/Dockerfile
context: ../ context: ../
image: commapp/identity-server:0.1 image: commapp/identity-server:0.1
container_name: identity-server container_name: identity-server
ports: ports:
- "${COMM_SERVICES_PORT_IDENTITY}:50051" - "${COMM_SERVICES_PORT_IDENTITY}:50051"
# localstack
localstack:
image: localstack/localstack
container_name: localstack
hostname: localstack
ports:
- "4566:4566"
environment:
- SERVICES=s3,dynamodb
- DATA_DIR=${TMPDIR}/localstack/data
- HOSTNAME_EXTERNAL=localstack
volumes:
jimUnsubmitted
Not Done
Inline Actions

Why bind mount a temp directory instead of using a data volume? It's just gonna create issues with permissions. Also ${TMPDIR} is not used to refer to a path in both the host and the container which is a bad idea.

jim: Why bind mount a temp directory instead of using a data volume? It's just gonna create issues…
- ${TMPDIR}:/tmp/localstack
networks:
- services-net