Skip to content

Add option for restriction of SELECT queries from user_model#17

Merged
xdaniel3 merged 1 commit intomasterfrom
tda/add_user_model_columns_option
Jan 11, 2026
Merged

Add option for restriction of SELECT queries from user_model#17
xdaniel3 merged 1 commit intomasterfrom
tda/add_user_model_columns_option

Conversation

@xdaniel3
Copy link

@xdaniel3 xdaniel3 commented Jan 5, 2026

  • in case that user_model is very wide this can save some resources

@xdaniel3 xdaniel3 force-pushed the tda/add_user_model_columns_option branch 2 times, most recently from 3471973 to 283e10d Compare January 5, 2026 14:22
@coveralls
Copy link

coveralls commented Jan 5, 2026

Pull Request Test Coverage Report for Build 20899885394

Details

  • 19 of 21 (90.48%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.0%) to 83.444%

Changes Missing Coverage Covered Lines Changed/Added Lines %
chamber/multidomains/domain.py 11 13 84.62%
Totals Coverage Status
Change from base Build 18522579058: 1.0%
Covered Lines: 1260
Relevant Lines: 1510

💛 - Coveralls

'http://localhost:8000/current_time_backend/')
assert_equal(reverse('current-datetime', site_id=settings.FRONTEND_SITE_ID, add_domain=True),
'https://localhost/current_time_frontend/')
def test_get_user_uses_only_when_columns_specified(self):
Copy link
Collaborator

Choose a reason for hiding this comment

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

nitpick: the name is slightly confusing, if i didn't know the usage of user_model_columns causes call of only method on the queryset I might have felt the name is missing a noun 😄

- in case that user_model is very wide this can save some resources
@xdaniel3 xdaniel3 force-pushed the tda/add_user_model_columns_option branch from 283e10d to e2d8c1b Compare January 11, 2026 18:30
@xdaniel3 xdaniel3 merged commit 0f5c6db into master Jan 11, 2026
3 checks passed
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.

3 participants