MerginMaps / MerginMaps/mobile
Handling of incompatible server version is buggy
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 388
- Forks
- 87
- Avg merge
- 2d 7h
- Merged PRs (30d)
- 20
Description
There are few issues related to handling of incompatible server version (e.g. if someone runs Mergin-CE with the default "dev" version instead of a proper version such as "2021.6.1"):
-
the warning text should be much more prominent and it should not be possible to log in. Currently logging in is possible, but then listing of projects seems not working (but it is just disabled) as well as other functionality gets disabled
-
the warning should add some more context - what is the server version, what is the minimum required version by Input
-
maybe disable version checking if the version is "dev" and assume it is the latest (?)
-
the warning text is overlapping with Mergin logo
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
No file or test is named in the issue. Start by locating the mobile app's server-version compatibility handling and reproduce the behavior with a server reporting the default "dev" version. Done means the incompatible-version warning is prominent, does not overlap the Mergin logo, includes both relevant versions, and prevents unusable login behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- mobile
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100