Skip to content

Conversation

@priscilawebdev
Copy link
Member

@priscilawebdev priscilawebdev commented Jan 16, 2025

closes #79395

Instead of including the settings select component directly in the onboarding documents, I propose adding a section to the unreal, unity, and minidumps platforms. This section will inform users about the option to "Store Minidumps as Attachments" and provide a link to the project settings where they can adjust this configuration.

Why this approach?

  • After checking, I found that we currently don’t offer a way to update settings directly in the onboarding docs. I’m concerned that adding this exception could lead to others replicating it in the future
  • Since storing minidumps is crucial for the success of these projects, I’ve added tracking to monitor if users click the link. If they don’t, we can then introduce the settings selection and evaluate if it improves the situation.

Preview
Screenshot 2025-01-16 at 15 23 20

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 16, 2025
@priscilawebdev priscilawebdev marked this pull request as ready for review January 16, 2025 14:51
@priscilawebdev priscilawebdev requested a review from a team as a code owner January 16, 2025 14:51
@priscilawebdev
Copy link
Member Author

priscilawebdev commented Jan 16, 2025

In the case of the unity platform, we also display a different UI on the issue stream page and it seems it is suppose to be as simple as possible as it does not show all the info we expose in the onboarding docs. would the callout in onboarding docs be enough?

Screen.Recording.2025-01-16.at.15.53.39.mov

@codecov
Copy link

codecov bot commented Jan 16, 2025

Bundle Report

Changes will increase total bundle size by 12.41kB (0.04%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
app-webpack-bundle-array-push 31.27MB 12.41kB (0.04%) ⬆️

@priscilawebdev priscilawebdev marked this pull request as draft January 17, 2025 15:38
@codecov
Copy link

codecov bot commented Jan 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #83577      +/-   ##
==========================================
- Coverage   87.56%   87.54%   -0.02%     
==========================================
  Files        9488     9409      -79     
  Lines      538571   537733     -838     
  Branches    21183    21179       -4     
==========================================
- Hits       471609   470774     -835     
+ Misses      66614    66611       -3     
  Partials      348      348              

@priscilawebdev
Copy link
Member Author

closed in favor of #83642

@github-actions github-actions bot locked and limited conversation to collaborators Feb 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Onboarding: Better communicate the ability of persisting minidumps

2 participants