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

Make avatars helper method simpler #16480

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

danidoni
Copy link
Contributor

@danidoni danidoni commented Jul 11, 2024

By moving code related to painting tags into the view, which is better at painting tags, the code is smaller

Depends on #16477

@github-actions github-actions bot added Frontend Things related to the OBS RoR app Test Suite / CI 💉 Things related to our tests/CI labels Jul 11, 2024
@danidoni danidoni force-pushed the make-avatars-helper-method-simpler branch 2 times, most recently from b783ace to cdee15a Compare July 11, 2024 11:50
By moving code related to painting tags into the view, which is better
at painting tags, the code is smaller
@saraycp saraycp force-pushed the make-avatars-helper-method-simpler branch from cdee15a to 6412628 Compare July 12, 2024 11:20
@saraycp
Copy link
Contributor

saraycp commented Jul 12, 2024

Rebased on master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend Things related to the OBS RoR app Test Suite / CI 💉 Things related to our tests/CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants