Skip to content

Update ConfigOptions.ts#4081

Closed
escix wants to merge 1 commit into
element-hq:livekitfrom
escix:patch-3
Closed

Update ConfigOptions.ts#4081
escix wants to merge 1 commit into
element-hq:livekitfrom
escix:patch-3

Conversation

@escix

@escix escix commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Created by Claude
Update to increase the timeout so element-call push notification arrive on mobile clients before timeout.

Important

Features and UI changes require a pre-approved issue.
Every PR must have a linked issue
that a maintainer has reviewed and approved before you started writing code.
PRs that don't meet this requirement will not be reviewed.
See CONTRIBUTING.md for ElementCall decided for this approach.

Content

Motivation and context

Screenshots / GIFs

Tests

  • Step 1
  • Step 2
  • Step ...

Checklist

  • A linked, pre-approved issue exists for this feature or UI change.
  • I have read CONTRIBUTING.md in full.
  • Pull request includes screenshots or videos for any UI changes.
  • Tests written for new code (and existing touched code where feasible).
  • Linter and other CI checks pass.
  • I have licensed the changes to Element by completing the Contributor License Agreement (CLA)

Update to increase the timeout so element-call push notification arrive on mobile clients before timeout.
@escix escix marked this pull request as ready for review July 3, 2026 04:36
@escix escix requested a review from a team as a code owner July 3, 2026 04:36
@escix escix requested a review from robintown July 3, 2026 04:36
@Johennes

Johennes commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Thanks for contributing this. Unfortunately, it doesn't follow our contribution guidelines.

Every PR must have a linked issue that a maintainer has reviewed and approved before you started writing code.

The change also doesn't appear to do anything other than add an unused property on ConfigOptions.

Therefore, I'll close this. If you're interested in continuing to work on this, please open an issue to discuss the changes you'd like to make.

@Johennes Johennes closed this Jul 3, 2026
@Johennes Johennes mentioned this pull request Jul 3, 2026
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants