Skip to content

Fixing check translation warning - #1522

Open
fateme-ramezanpour wants to merge 3 commits into
masterfrom
fixing-check-translation-warning
Open

Fixing check translation warning#1522
fateme-ramezanpour wants to merge 3 commits into
masterfrom
fixing-check-translation-warning

Conversation

@fateme-ramezanpour

@fateme-ramezanpour fateme-ramezanpour commented Apr 9, 2025

Copy link
Copy Markdown
Contributor

Description

Fixes a runtime warning in the CheckTranslation component on the Edit Project page.
The warning was due to an invalid prop or unexpected data passed during rendering.

Screenshots

Screenshot 2025-04-09 at 5 12 05 PM
Screenshot 2025-04-09 at 4 52 45 PM

Checked the following

  • Pages affected by this change work on mobile
  • Pages affected by this change work when logged out
  • Pages affected by this change work when logged in
  • Pages affected by this change work with custom theme and default theme
  • Run within /frontend: yarn format && yarn lint
  • Run within /backend: make format && make lint

What and Why

@fateme-ramezanpour
fateme-ramezanpour marked this pull request as ready for review April 9, 2025 15:24
@fateme-ramezanpour fateme-ramezanpour added the PR: looking for reviewer PR is ready for a review, but no reviewer has been appointed label Apr 9, 2025
@fateme-ramezanpour fateme-ramezanpour self-assigned this Apr 9, 2025
@Idontker Idontker added PR: in progress PR is currently being reviewed or a reviewer has been selected and removed PR: looking for reviewer PR is ready for a review, but no reviewer has been appointed labels Apr 10, 2025
@Idontker
Idontker requested review from Idontker and removed request for Idontker April 10, 2025 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: in progress PR is currently being reviewed or a reviewer has been selected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants