Skip to content
This repository has been archived by the owner on Sep 3, 2023. It is now read-only.

Add support for 'About Libraries' Android library #1411

Closed
wants to merge 3 commits into from
Closed

Add support for 'About Libraries' Android library #1411

wants to merge 3 commits into from

Conversation

AllanWang
Copy link

@AllanWang AllanWang commented Jul 23, 2017

This is to support About Libraries, a library to facilitate listing dependencies. Your project is currently embedded in that project, which means the version number needs to be manually changed. Having it in your project directly along with the resValue in gradle will make this automatic.

These resources will be removed if users have proguard. Those who choose to use about libraries with proguard will keep the R classes to resolve that.

Attached below is an example of how it looks. (Note that the UI can be customized, but the text will be the same throughout projects)

screenshot_20170723-123845

<string name="library_materialdialogs_isOpenSource">true</string>
<string name="library_materialdialogs_repositoryLink">https://github.com/afollestad/material-dialogs</string>
<string name="library_materialdialogs_classPath">com.afollestad.materialdialogs.MaterialDialog</string>
<string name="library_materialdialogs_licenseId">mit</string>

Choose a reason for hiding this comment

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

Shouldn't it be uppercase?

Copy link
Author

Choose a reason for hiding this comment

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

@piotrek1543 Are you referring to the materialdialogs portion of the string name? It doesn't actually matter for developers as the strings are converted to all lower case when they request it. If you'd like I can make them upper case.

Choose a reason for hiding this comment

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

@AllanWang i was curious about why 'mit' is lowercase. No, I don't see this change as necessary

Copy link
Author

Choose a reason for hiding this comment

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

@piotrek1543 these strings were done by another user and I just copied them over. mit is a keyword used by the library to generate the license. You can see in the screenshot (which is updated) that it becomes "MIT License".

If there are any issues in the description of the app that can be updated. (Again, written by another user based on how MD was at the time)

@piotrek1543
Copy link

Update please screenshot - add one with right API value

@piotrek1543
Copy link

Change also this misleading MR name - it's more for supporting another library than just creating a new about file.

Maybe: 'Adding support for 'About Libraries' Android library'

Then you would have my approval

@AllanWang AllanWang changed the title Create about library file Add support for 'About Libraries' Android library Jul 23, 2017
@piotrek1543
Copy link

@AllanWang lol, it seems that only author of this repo can review this correctly and then merge.

For me it's nice-to-have feature, so... approve.

@AllanWang
Copy link
Author

@piotrek1543 That's fine. Thanks for the approval

@afollestad afollestad closed this May 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants