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

Streamline and align the "Getting Started" docs to fit in the new structure #1585

Closed
Tracked by #23
priscilawebdev opened this issue Aug 4, 2023 · 6 comments
Closed
Tracked by #23

Comments

@priscilawebdev
Copy link
Member

priscilawebdev commented Aug 4, 2023

Problem Statement

  • The "Getting Started" documentation for all sentry-dart platforms may not conform to the newly defined format, which includes the steps "Install", "Configure", "Verify", and the optional "Next Steps".
    • The "Verify" step is not always available and it is a very important step that helps the user to verify that the installation and configuration were done correctly.
  • New users onboarding with Sentry may find the existing documentation extensive and unclear. The "Getting Started" guide should cover the basics, while more advanced configurations shall live on the sentry-docs website.
  • Some copies and code snippets in the documentation may be outdated.

Solution Brainstorm

The task goals are as follows:

  • Thoroughly review all sentry-dart platform documentation, including less commonly used sections, and reorganize their content into structured headings: 'Install,' 'Configure,' 'Verify,' and the optional 'Next Steps.'
  • The onboarding documentation should strictly focus on the basics to help users get started. Advanced configurations should be available in the complete documentation on the sentry-docs website.
  • Essential information for new users should be included under the new 'Notes' section in the onboarding documentation.
  • Update code snippets and copies as necessary.
  • If a "Verify" step is missing, include it in the documentation.

Please refer to the "Frontendend Handbook" guide in our "Sentry" main repository for assistance in adding and updating the getting started docs: Frontendend Handbook.

Project: getsentry/team-sdks#23

@denrase
Copy link
Collaborator

denrase commented Oct 3, 2023

I looked through everything. dart and flutter are already in the list of migrated frameworks. They follow the new format, have sample code & icons. So everything should be there, at least as far as I can tell.

The only thing that dart & flutter do is they have extra sections after Verify, which are on the same hierarchy level, for example a description on how to setup Performance. Is this something we want to remove?

Only discovered a small improvement in the dart getting started sample, for which i created a PR: getsentry/sentry#57348

@kahest @priscilawebdev Is there anything else that needs to be done to close this issue?

@kahest
Copy link
Member

kahest commented Oct 3, 2023

@ArthurKnaus @denrase currently looks like this
getting-started-dart
getting-started-flutter

To adhere to the generic structure, should the current sections Performance (Dart+Flutter) and Debug Symbols + Source Context (Flutter only) should go under a new section Next Steps?

@ArthurKnaus
Copy link
Member

@kahest @denrase
I am not familiar with the mobile SDK functionality. If this is required to get performance measurements / if there is no automatic instrumentation, I would keep it in there. Otherwise move it to "next steps".
If it makes sense please also move "Debug Symbols" and "Source context" to the next steps. We only want the minimal information to get the user started in these docs.

@kahest
Copy link
Member

kahest commented Oct 3, 2023

@ArthurKnaus Thanks for clarifying - there is no monkeypatching mechanism or similar in Flutter/Dart, so to get perf instrumentation, the user needs to at least add minimal init/wrapping, see relevant docs.

@denrase My take is then that we should keep the Performance section for Flutter+Dart, but we can move Debug Symbols and Source Context under a Next Steps section similar to here on javascript:

getting-started-browser-javascript

@kahest
Copy link
Member

kahest commented Nov 6, 2023

@buenaflor I assume we can close this?

@denrase
Copy link
Collaborator

denrase commented Nov 6, 2023

@kahest Yeas, the relevant PRs have been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

4 participants