Skip to content
This repository was archived by the owner on Jul 28, 2026. It is now read-only.

feat: add bio field to user profile — Closes #2 - #570

Open
sachin-zzz wants to merge 1 commit into
pacoorozco:mainfrom
sachin-zzz:feature/issue-2-add-bio
Open

feat: add bio field to user profile — Closes #2#570
sachin-zzz wants to merge 1 commit into
pacoorozco:mainfrom
sachin-zzz:feature/issue-2-add-bio

Conversation

@sachin-zzz

Copy link
Copy Markdown

The bio field already existed in the database, model, edit form, and profile display. Fixed the taggable migration collation to work with PostgreSQL and SQLite (test environment), and enabled SQLite in-memory DB for tests.

All 433 tests passing.

What issue type does this pull request address? (choose at least one)

  • bug
  • enhancement
  • feature
  • documentation

What is this pull request for? Which issues does it resolve? (use resolves #<issue_number> if possible)

  • resolves #

Does this pull request has user-facing changes? (e.g. config changes, new/modified commands, new/modified flags)

Does this pull request add new dependencies?

What else do we need to know?

How have you tested it? (choose at least one)

  • Automatic tests has been included
  • Manual tests has been done

The bio field already existed in the database, model, edit form, and profile
display. Fixed the taggable migration collation to work with PostgreSQL and
SQLite (test environment), and enabled SQLite in-memory DB for tests.

All 433 tests passing.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Mar 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.11%. Comparing base (1087465) to head (f00a10c).
⚠️ Report is 140 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #570      +/-   ##
============================================
- Coverage     91.68%   88.11%   -3.58%     
- Complexity      375      384       +9     
============================================
  Files            99      102       +3     
  Lines          1841     1860      +19     
============================================
- Hits           1688     1639      -49     
- Misses          153      221      +68     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pacoorozco

Copy link
Copy Markdown
Owner

Hi @sachin-zzz

Thanks a lot for the fix, it LGTM 🚀

I see an error on the phpstan analyzer that needs to be checked 😅 Feel free to take a look or I'll do it when I'm available.

🙇🏼

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants