Skip to content

Fix API fields filtering - #8102

Open
Guflly wants to merge 1 commit into
MarkUsProject:masterfrom
Guflly:fix/api-fields-filtering
Open

Fix API fields filtering#8102
Guflly wants to merge 1 commit into
MarkUsProject:masterfrom
Guflly:fix/api-fields-filtering

Conversation

@Guflly

@Guflly Guflly commented Aug 2, 2026

Copy link
Copy Markdown

Proposed Changes

Use the requested fields list when rendering user and course API responses instead of always returning every default field.

Closes #8071

Screenshots of your changes (if applicable)

Not applicable.

Type of Change

Type Applies?
Bug fix X

Checklist

Before opening your pull request:

  • I have performed a self-review of my changes.
  • I have added tests for my changes, if applicable.
  • I have updated the project documentation, if applicable.
  • If this is my first contribution, I have added myself to the list of contributors.

After opening your pull request:

  • I have updated the project Changelog.
  • I have verified that the pre-commit.ci checks have passed.
  • I have verified that the CI tests have passed.
  • I have reviewed the test coverage changes reported by Coveralls.
  • I have requested a review from a project maintainer.

Questions and Comments

Local checks: RuboCop on the changed Ruby files and Ruby syntax checks. The controller specs are included for CI.

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.

API "Common optional parameters" fields not working

1 participant