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

[flutter_local_notifications] Reorganize the readme in to smaller digestible blocks #2308

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

davetrollope
Copy link

When I tried to adopt the flutter_local_notifications package, I found it difficult to work through the readme. This PR proposes isolating the larger platform specific blocks in to separate files making the overall readme easier to follow.

I have updated to include the latest 17.1.0 updates to the README.

@MaikuB
Copy link
Owner

MaikuB commented May 1, 2024

Thanks for the PR. I understand the reason behind the changes and for it but will take me some time to digest to see how the changes work. Do you by chance know how this will work on pub.dev and if there similar examples on pub.dev? Mentioning it as a readme will show up on the readme section and I don't know what would happen if a user was to click on, say, the link to the Android markdown file

@davetrollope
Copy link
Author

I am not sure about pub.dev, but I think it follows standard markdown. I will investigate and see if I can find some examples to verify.

@davetrollope
Copy link
Author

After doing some research, it looks like relative links are expected to be rewritten to github URLs. That means following the links would hop to the github view of the page, e.g. https://github.com/MaikuB/flutter_local_notifications/blob/49dc49d7ab19ceb6afcf99bfb84d5807c515aaf7/flutter_local_notifications/Android.md

I'm in two minds about this.

  1. It might be a bit disconcerting to be forced over to github for subsections, and then have to hit back or close the tab to continue the overall flow of setup - at least as the subpages are written right now. Perhaps some nuanced updates recognizing this would make it intuitive.
  2. perhaps keeping everything in one readme is better, but maybe we should reorganize the file in to complete sections/walkthroughs for a platform (perhaps with a badge that identifies common setup?

Open to thoughts and ideas.

@MaikuB
Copy link
Owner

MaikuB commented May 9, 2024

Thanks for investigating. I agree that it's disconcerting to be taken elsewhere whilst in the middle of reading instructions. I remembered a while back I was notified about https://docs.page/ where following a convention using that domain allows content to be pulled from repo (reference https://invertase.io/blog/introducing-docs-page). Perhaps this an opportunity create the docs folder and move the content of the readme that has been split there and change the readme to have a link that says go to docs.page/MaikuB/flutter_local_notifications for more info for more details on how to use the plugin?

@davetrollope
Copy link
Author

Yes, I like the idea of that. I will start playing with it.

@Levi-Lesches
Copy link
Contributor

I second the idea of a smaller main README linking to other pages with more details. Lets readers understand the bigger picture faster while still getting to choose when to zoom in on a specific platform, function, etc.

@Levi-Lesches
Copy link
Contributor

@MaikuB I think this should be re-prioritized as a lot of the issues opened recently are the result of people not reading the README. I understand that's just part of the human condition, but a much shorter and platform-specific guide should help

@Levi-Lesches
Copy link
Contributor

I started work at #2477 to not only separate the content by platform, but also reorganize each topic, rewrite it for clarity, include more links, more full examples, and removed content that applies to Flutter or Android versions this package no longer supports.

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.

4 participants