Skip to content
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.

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

iOS AppStore release 4.0.4 #113

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

iOS AppStore release 4.0.4 #113

wants to merge 16 commits into from

Conversation

magicwave
Copy link
Collaborator

DO NOT MERGE TO MAIN

UI updates for AppStore release: positioning of the camera button in the chat thread view to conform with Apple's HIG.

This is a special release to get UI updates into the AppStore before merging the DQL migration release, which will be a breaking change.

This release branch is cut from legacy-builder-api branch for backward compatibility with the previous AppStore release which was cut from legacy-builder-api, which was created to preserve a legacy builder version of the codebase after the DQL migration was merged to main.

The UI update commits were merged to main with the DQL migration branch still independent, but then it was noticed that main and legacy-builder-api had diverged, and the 4.0.3 AppStore release was no longer compatible with main. So the UI update commits were cherry-picked from main to legacy-builder-api and this release cut from that.

From here, the legacy-builder-api branch should be merged to DQL branch and thoroughly tested along with the Android version in this same repo. Only after QA is satisfied on and between both platform versions should DQL be merged to main. From main then can be cut v5.0.0, a major, backwards-incompatible, release.

texasRanger09 and others added 16 commits February 27, 2024 14:19
Android - Remove unused/broken functionality
Removed the twitter profile section/link (Out of scope and didn't work) as well as the ability to message someone from the profile page as this functionality was broken and out of scope.
When cursor positioned at start of chat input text, cursor was too far
to leading edge.

- add small edge inset margin to chat input view, ExpandingTextView wrapper
@magicwave
Copy link
Collaborator Author

The release issue #112 is related to this PR. (It appears to not be linkable through the Github UI, maybe because there is a merge conflict; however the conflict will not be resolved since this branch should not be merged.

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.

None yet

2 participants