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

Fix user bios job in the case that user_bio is None #3632

Merged
merged 1 commit into from
Jul 9, 2017
Merged

Conversation

wxyxinyu
Copy link
Contributor

@wxyxinyu wxyxinyu commented Jul 9, 2017

We have some legacy data where the user_bio field of the UserSettingsModel for some older users can be None. This PR is to handle that case.

Copy link
Contributor

@shaz13 shaz13 left a comment

Choose a reason for hiding this comment

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

screenshot from 2017-07-09 20-22-18

Tests passed successfully. Thanks @wxyxinyu
LGTM!

@shaz13 shaz13 added the PR: LGTM label Jul 9, 2017
@codecov-io
Copy link

Codecov Report

Merging #3632 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #3632   +/-   ##
========================================
  Coverage    44.84%   44.84%           
========================================
  Files          257      257           
  Lines        19873    19873           
  Branches      3132     3132           
========================================
  Hits          8913     8913           
  Misses       10960    10960

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 aac1565...c0c8ed3. Read the comment docs.

@wxyxinyu wxyxinyu merged commit c148c77 into develop Jul 9, 2017
@wxyxinyu wxyxinyu deleted the user-bios-job branch July 9, 2017 17:00
giritheja added a commit to giritheja/oppia that referenced this pull request Jul 16, 2017
…date

* upstream/develop: (32 commits)
  Remove erroneous clause in app.yaml. (oppia#3643)
  Part of oppia#2447: Upgrade select2 library (oppia#3626)
  Fix part of oppia#2394: Added docstrings in exp_domain.py (oppia#3578)
  Fix oppia#3618: stop icon overlap (oppia#3642)
  Add backend functionality to correctly handle needs_update part of subtitled HTML. (oppia#3620)
  Change 'style' to 'ng-attr-style' in an attempt to fix IE bug in collection viewer. (oppia#3636)
  Fix typo in Exploration editor (oppia#3637)
  Update credits (oppia#3633)
  Fix oppia#2639: Updated Google chart/visualization Library loader code and use async to load the library (oppia#3614)
  Fix user bios job in the case that user_bio is None (oppia#3632)
  Update changelog and credits for release 2.5.2 (oppia#3628)
  Fix oppia#2394: Add docstring to core.domain.user_jobs_continous.py (oppia#3569)
  Fix oppia#3585: Corrected arrows in exploration progress nav (oppia#3605)
  change the text
  edit error message
  Corrected tense of error message
  Corrected state to read card
  Close modal window if clicked outside (oppia#3623)
  Storage, Domain classes and helper functions for ClassifierExplorationMapping (oppia#3583)
  removed super admin role from admin interface. (oppia#3621)
  ...
ctao5660 pushed a commit to ctao5660/oppia that referenced this pull request Nov 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants