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.

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

chore(release): 8.57.0 release candidate [DO-NOT-MERGE] #11189

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

chore(release): 8.57.0 release candidate [DO-NOT-MERGE] #11189

wants to merge 14 commits into from
+166 −205

Conversation

MrSltun
Copy link
Member

@MrSltun MrSltun commented Nov 25, 2024

fix: alert edit bottom sheet visible

This PR resolves []

Description

PR Checklist

  • I have tested my changes on iOS and Android.
  • I hid my changes behind a feature flag, or they don't need one.
  • I have included screenshots or videos, or I have not changed the UI.
  • I have added tests, or my changes don't require any.
  • I added an app state migration, or my changes do not require one.
  • I have documented any follow-up work that this PR will require, or it does not require any.
  • I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

  • I would like at least one of the reviewers to run this PR on the simulator or device.

#nochangelog

Need help with something? Have a look at our docs, or get in touch with us.

@gkartalis @MrSltun
fix: alert edit bottom sheet cut off (#11180)
8943e02 
fix: alert edit bottom sheet visible
@MrSltun MrSltun self-assigned this Nov 25, 2024
dariakoko and others added 13 commits November 25, 2024 16:34
@dariakoko @MrSltun
chore: set AREnableNewSaveAndFollowOnArtworkCard ready for release (#…
d1c401b 
…11185)

chore: set AREnableNewSaveAndFollowOnArtworkCard readty for release
@dariakoko @MrSltun
fix: adjust SellWithArtsyRecentlySold rail card height (#11188)
2355a34
@MounirDhahri @MrSltun
fix: broken bottom tabs on tablets (#11198)
507b2f1 
# Conflicts:
#	src/app/Navigation/AuthenticatedRoutes/Tabs.tsx
@MounirDhahri @MrSltun
fix: back navigation on android (#11202)
3aaed8e
@MounirDhahri @MrSltun
fix: sort and filter button on auction screen (#11203)
5b4f955 
# Conflicts:
#	src/app/AppRegistry.tsx
@MounirDhahri @MrSltun
fix: broken back button on terms (#11199)
d5431ea
@MrSltun
fix broken tests
ebe8087
@MounirDhahri @MrSltun
fix: broken new search tests (#11204)
70e4513
@MounirDhahri @MrSltun
fix: broken my-profile tap after purchase (#11200)
6f29138 
* fix: broken my-profile tap after purchase

* fix: again

* fix: again 2

* chore: update routes.tests.ts

* fix: typo
# Conflicts:
#	src/app/AppRegistry.tsx
#	src/app/system/navigation/navigate.ts
@MounirDhahri @MrSltun
fix: sort & filter modal performance issues (#11213)
5627a26 
fix: sort & filter issues
@gkartalis @MrSltun
fix: bottom sheet edit alert (#11216)
a49f5ca 
* fix: bottom sheet edit alert

* chore: comment

* chore: comment
@MrSltun
fix: Update Artsy Guarantee position in Conversation (#11219)
caff1a0
@MrSltun
fix: Revert: "fix: deep link analytics event triggered twice" (#11221)
e02381a 
Revert "fix: deep link analytics event triggered twice (#11110)"

This reverts commit 1cee013.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@MrSltun MrSltun

Labels
Do not merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MrSltun @gkartalis @dariakoko @MounirDhahri