Skip to content

Remove Logs from Release Build#66

Merged
nyerunkar merged 3 commits intomainfrom
access-control-page-ui-updation
Mar 12, 2026
Merged

Remove Logs from Release Build#66
nyerunkar merged 3 commits intomainfrom
access-control-page-ui-updation

Conversation

@girishjadhav-cosmica
Copy link
Member

Summary

This MR ensures that application logs are visible only in Debug builds and completely removed from Release builds.

Changes Made

  • Wrapped logging statements using conditional compilation (#if DEBUG).
  • Ensured no print or debug logs are executed in the Release configuration.
  • Verified logging behavior in both Debug and Release builds.

Testing

  • Ran the application in Debug build and confirmed logs appear in the console.
  • Ran the application in Release build and confirmed no logs are printed.

Expected Outcome

  • Logs will assist developers during development.
  • Production (Release) builds will have no console logs, improving security and performance.

Notes

This change prevents accidental exposure of internal debug information in production builds.
Closes #65

…ccess, and file paste) #61

- Disable Share Extension activation so the app does not appear in the iOS Share Sheet
- Disable external document access via Files app (UIFileSharingEnabled=false, LSSupportsOpeningDocumentsInPlace=false)
- Block image/file paste from clipboard in chat input
- Allow text paste to continue working normally

Tests:
- Verify app does not appear in iOS Share Sheet
- Verify Files app cannot open files in the app
- Verify image/file paste from clipboard is blocked
- Verify text paste continues to work

This ensures files cannot be imported into the app from outside sources.
… and light theme UI #63

- Show owner information in Peer1 without status or permission icons
- Display shared user information in Peer2 with download/forward permissions
- Add access status messages (Access Until, File Expired, Access Revoked, Access Denied)
- Add forwarded users section
- Implement search functionality for forwarded users
- Show search bar only when forwarded users count is greater than 10
- Hide owner and shared sections during search
- Show "No Forwarded User found with <name>" when search has no results
- Append "(You)" label for owner (Peer1) and shared user (Peer2)
- Truncate long usernames while preserving "(You)" label
- Fix light theme visibility issues (back icon, search text, hint text, close icon)

Test:
- Verified owner information displays correctly in Peer1 without status icons
- Verified shared users show correct permissions and access status in Peer2
- Confirmed forwarded users appear in a separate section
- Verified search bar appears only when forwarded users count is greater than 10
- Tested forwarded user search functionality and empty state message
- Verified "(You)" label displays correctly even with long usernames
- Checked UI visibility in both Light and Dark themes
test: verified logs are visible only in Debug build and not in Release build
issur ref #65
Copy link
Collaborator

@nyerunkar nyerunkar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@nyerunkar nyerunkar merged commit 755f11b into main Mar 12, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove Logs from Release Build

2 participants