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

7.0.0 #2958

Merged
merged 55 commits into from
Nov 29, 2023
Merged

7.0.0 #2958

merged 55 commits into from
Nov 29, 2023

Conversation

romtsn
Copy link
Member

@romtsn romtsn commented Sep 28, 2023 •
edited
Loading

#skip-changelog

📜 Description

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

  • I reviewed the submitted code.
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

markushi and others added 27 commits June 29, 2023 14:14
@markushi
Move enableNdk to SentryAndroidOptions (#2793)
0566579
@markushi
Capture failed HTTP requests by default (#2794)
4f55fd9
@markushi
Merge branch 'main' into feat/7.0.0
d6fc22e
@markushi
Fix Changelog
36afdd4
@markushi
Merge branch 'feat/7.0.0' of github.com:getsentry/sentry-java into fe…
590fcb8 
…at/7.0.0
@romtsn
7.0.0 -> Unreleased in changelog
1faf168
@romtsn
Do not overwrite UI transaction status if set by the user (#2852)
cae0eea
@markushi
Reduce flush timeout to 4s on Android to avoid ANRs (#2858)
3838091
@markushi
Set ip_address = {{auto}} by default (#2860)
19fca04
@romtsn
Merge branch 'main' into feat/7.0.0
6182cc2
@romtsn
Fix test
7717683
@romtsn
Measure AppStart time till First Draw (#2851)
1c0c691
@lbloder @getsentry-bot @romtsn
update coroutines to 1.6.1, use CopyableThreadContextElement (#2838)
803f03c 
Co-authored-by: Sentry Github Bot <[email protected]>
Co-authored-by: Roman Zavarnitsyn <[email protected]>
@stefanosiano
Reduce timeout of AsyncHttpTransport to avoid ANR (#2879)
8e78ac3 
* reduced timeout of AsyncHttpTransport close to flushTimeoutMillis (default 4s in Android and 15s in Java) to avoid ANRs in Android
@markushi
Add deadline timeout for automatic transactions (#2865)
0b3de21 
* Add deadline timeout for automatic transactions

* Update Changelog

* Address PR comments

* Fix formatting

* Add missing test, improve naming

* Ensure deadline timeout is only set once
@markushi
Apollo v2 BeforeSpanCallback: Allow returning null (#2890)
ee21d53
@markushi
Start a new automatic transaction on every click (#2891)
55b103c
@stefanosiano
Bump min API to 19 (#2883)
b0e93e8 
* updated minimum Android SDK version to 19
* removed few workarounds for version < 19
@markushi
Fix don't overwrite the span status of unfinished spans (#2859)
bdf1379
@markushi
(Android) Use root transaction instead of last active span (#2855)
cd268a3
@romtsn
Merge branch 'main' into feat/7.0.0
1821d66
@markushi
Fix changelog
421d724
@romtsn
Make SDK unity-compatible (#2847)
c383ed1
@romtsn
Capture unfinished transactions on crash (#2938)
72e23fc
@markushi @romtsn @getsentry-bot
Observe network state to upload any unsent envelopes (#2910)
d6cd78c 
Co-authored-by: Roman Zavarnitsyn <[email protected]>
Co-authored-by: Sentry Github Bot <[email protected]>
@romtsn
Merge branch 'main' into feat/7.0.0
227698f
@romtsn
Highlight breaking chnages in changelog
ca595ea
@github-actions
Copy link
Contributor

github-actions bot commented Sep 28, 2023 •
edited
Loading

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 971b171

@romtsn romtsn marked this pull request as ready for review November 27, 2023 16:13
kahest
kahest reviewed Nov 27, 2023
View reviewed changes
Copy link
Member

@kahest kahest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great update, and great release notes 🥔🥔🥔🥔🥔
left some thoughts that also apply to the migration docs

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@romtsn romtsn mentioned this pull request Nov 27, 2023
Merged
3 tasks
lbloder and others added 3 commits November 28, 2023 17:03
@lbloder @getsentry-bot
Always execute callback in withScope (#3066)
741b219 
* execute withScopeCallback in noopHub, add tests to SentryTest

* extract IScope interface

* initial migration to IScope

* fix usages of IScope constructor and replace with Scope, add fromScope static Method, fix tests, add tests, run withScope even on disabled hub

* replace static fromScope method with clone

* add changelog entries

* revert unintended changes

* Format code

---------

Co-authored-by: Sentry Github Bot <[email protected]>
@romtsn
Mention startTransaction overloads removal
9f185cb
@romtsn
Fix was not finishing span when it was dropped
971b171
@romtsn romtsn merged commit 8fd337b into main Nov 29, 2023
18 of 19 checks passed
@romtsn romtsn deleted the feat/7.0.0 branch November 29, 2023 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kahest kahest kahest left review comments

@markushi markushi markushi approved these changes

@adinauer adinauer Awaiting requested review from adinauer adinauer is a code owner

@stefanosiano stefanosiano Awaiting requested review from stefanosiano stefanosiano is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@romtsn @markushi @kahest @lbloder @stefanosiano @getsentry-bot @adinauer @ToppleTheNun