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

refactor: removed manifest warnings #433

Merged
merged 1 commit into from
Aug 20, 2019

Conversation

CloudyPadmal
Copy link
Contributor

Fixes #432

Changes:

  • Added ignore clauses for backup and app indexing

Screenshot/s for the changes:

Not available

Checklist: [Please tick following check boxes with [x] if the respective task is completed]

  • No modifications done at the end of resource files strings.xml, dimens.xml or colors.xml
  • I have reformatted code in every file included in this PR [CTRL+ALT+L]
  • My code does not contain any extra lines or extra spaces
  • I have requested reviews from other members

APK for testing:

Not available

@CloudyPadmal CloudyPadmal added the enhancement Improvement to an existing feature label Aug 20, 2019
@CloudyPadmal CloudyPadmal self-assigned this Aug 20, 2019
Copy link
Contributor

@adityastic adityastic left a comment

Choose a reason for hiding this comment

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

@CloudyPadmal the error must be regarding not having ACTION-VIEW in your manifest. A better way to solve this is maybe by adding:
<action android:name="android.intent.action.VIEW" />
to the default activity

Copy link
Contributor

@adityastic adityastic left a comment

Choose a reason for hiding this comment

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

👍 Travis is having some issues maybe, not able to clone for some reason

@CloudyPadmal
Copy link
Contributor Author

🤔 there are extra builds.

@CloudyPadmal CloudyPadmal merged commit 8be5e64 into fossasia:development Aug 20, 2019
@CloudyPadmal CloudyPadmal deleted the manifest-warnings branch August 20, 2019 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Warnings in Android manifest file
2 participants