Skip to content

remove unused /share placeholder text from session header #74

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
ariane-emory wants to merge 12 commits into
base: dev
Choose a base branch
from
Draft

remove unused /share placeholder text from session header #74

ariane-emory wants to merge 12 commits into from

Conversation

@ariane-emory
Copy link
Owner

@ariane-emory ariane-emory commented Dec 25, 2025

DUMMY PR, IGNORE.

  • Remove '/share copy link' placeholder that appeared when session was not shared
  • Simplify header logic to only show share URL when session is actually shared
  • Remove unused shareEnabled memo variable

@ariane-emory
remove unused /share placeholder text from session header
1f2a1d4 
- Remove '/share copy link' placeholder that appeared when session was not shared
- Simplify header logic to only show share URL when session is actually shared
- Remove unused shareEnabled memo variable
@ariane-emory
Merge branch 'dev' into feat/no-share-in-header
8fc81ca
@ariane-emory
Merge branch 'dev' into feat/no-share-in-header
0952be2
@github-actions
Copy link

github-actions bot commented Jan 8, 2026

Hey! Your PR title remove unused /share placeholder text from session header doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

@ariane-emory ariane-emory force-pushed the feat/no-share-in-header branch from f97cd76 to 97905f5 Compare January 9, 2026 05:24
@ariane-emory ariane-emory force-pushed the feat/no-share-in-header branch 2 times, most recently from c423745 to 482f396 Compare January 10, 2026 22:37
@ariane-emory ariane-emory force-pushed the feat/no-share-in-header branch 2 times, most recently from f88b93a to 1ae2430 Compare January 13, 2026 06:12
@ariane-emory ariane-emory force-pushed the feat/no-share-in-header branch from 1ae2430 to 80f7b3c Compare January 13, 2026 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

needs:title

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ariane-emory