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

Fixed #1441 Edit event triggers to upload image from anywhere in the layout of edit event #1442

Merged
merged 1 commit into from
Jan 14, 2019

Conversation

HHimanshusahni
Copy link
Member

@HHimanshusahni HHimanshusahni commented Jan 14, 2019

Fixes #1441

Checklist:

  • I have checked for PMD and check-style issues
  • I have read the Contribution & Best practices Guide and my PR follows them.
  • My branch is up-to-date with the Upstream development branch.

Changes:

Upload Image option now gets trigger when we tap on the imageview of Update event and not from anywhere else in the layout.

@codecov
Copy link

codecov bot commented Jan 14, 2019

Codecov Report

Merging #1442 into development will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@              Coverage Diff               @@
##             development    #1442   +/-   ##
==============================================
  Coverage          24.25%   24.25%           
  Complexity           757      757           
==============================================
  Files                237      237           
  Lines               8736     8736           
  Branches             356      356           
==============================================
  Hits                2119     2119           
  Misses              6533     6533           
  Partials              84       84
Impacted Files Coverage Δ Complexity Δ
...ganizer/core/event/create/UpdateEventFragment.java 0% <0%> (ø) 0 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 527cf01...301b2e8. Read the comment docs.

@iamareebjamal iamareebjamal merged commit da4b930 into fossasia:development Jan 14, 2019
HHimanshusahni added a commit to HHimanshusahni/open-event-orga-app that referenced this pull request Jan 16, 2019
HHimanshusahni added a commit to HHimanshusahni/open-event-orga-app that referenced this pull request Jan 16, 2019
@HHimanshusahni HHimanshusahni deleted the bugfix_branch branch January 16, 2019 16:04
Masquerade0097 pushed a commit that referenced this pull request Feb 8, 2019
*  Fix:IllegalArgumentException: Unsupported ISO 3166 country

* fix: Edit event triggers to upload image from anywhere in the layout of edit event (#1442)

* chore(deps): bump library from 1.2.4 to 1.2.5 (#1444)

Bumps [library](https://github.com/Pes8/android-material-color-picker-dialog) from 1.2.4 to 1.2.5.
- [Release notes](https://github.com/Pes8/android-material-color-picker-dialog/releases)
- [Commits](https://github.com/Pes8/android-material-color-picker-dialog/commits)

Signed-off-by: dependabot[bot] <[email protected]>

* added checks to fix IllegalArgumentException

* This reverts commit 2d7d87d.

* Revert "fix: Edit event triggers to upload image from anywhere in the layout of edit event (#1442)"

This reverts commit 2869405.

* Fixed  the space around symbols and removed unnecessary blank lines

* Using country.equals()  method to perform the check ,which fixes the exception

* Made changes in the check statement

* Used Utils.isEmpty() method  in the check statement

* Removing country.equals() whichis redundant
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.

2 participants