Talk sidebar integration disabled on Maps app
Open
Nobody has claimed this yet.
feature: frontend 🖌️
feature: integration 📦
technical debt
- Dominant language
- PHP
- Stars
- 2.2k
- Forks
- 587
- Avg merge
- 17h 55m
- Merged PRs (30d)
- 350
Description
How to use GitHub
- Please use the 👍 reaction to show that you are affected by the same issue.
- Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
- Subscribe to receive notifications on status change and new comments.
Ref https://github.com/nextcloud/spreed/pull/8588 and https://github.com/nextcloud/maps/issues/947
The current solution is quite harsh, but was picked for practical reasons. Would be good to have a look why/where leaflet versions are destroying/interfering themselves
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the referenced spreed pull request #8588 and Maps issue #947, then trace the Talk sidebar integration and the Leaflet versions used by both applications. Done means identifying why the versions interfere and restoring the sidebar integration without the current harsh workaround.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100