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

Removed redundant fields in main subjects for PATCH user #3070

Open
wants to merge 17 commits into
base: develop
Choose a base branch
from

Conversation

Made1ra
Copy link
Contributor

@Made1ra Made1ra commented Jan 8, 2025

When users update their mainSubjects, only required fields are sent to the server.

Before:
image

After:
Screenshot 2025-01-30 at 14 04 01

@Made1ra Made1ra added bug Something isn't working Frontend part labels Jan 8, 2025
@Made1ra Made1ra self-assigned this Jan 8, 2025
@Made1ra Made1ra linked an issue Jan 8, 2025 that may be closed by this pull request
@Made1ra Made1ra force-pushed the bugfix/3067/fix-redundant-fields-in-main-subjects branch from 63aa622 to 36850da Compare January 9, 2025 00:47
@Made1ra Made1ra changed the title Removed redundant fields in main subjects Removed redundant fields in main subjects for PATCH user Jan 10, 2025
@Made1ra Made1ra requested a review from sandrvvu January 15, 2025 14:12
@Made1ra Made1ra requested a review from luiqor January 30, 2025 12:05
Copy link

sonarqubecloud bot commented Feb 4, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
67.3% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Frontend part
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix redundant fields in mainSubjects for PATCH user
3 participants