Skip to content

Trigger Fork CI - #4

Closed
Rohan-Unbeg wants to merge 35 commits into
developfrom
gsoc-m2-feedback-fixes
Closed

Rohan-Unbeg wants to merge 35 commits into
developfrom
gsoc-m2-feedback-fixes

Conversation

@Rohan-Unbeg

@Rohan-Unbeg Rohan-Unbeg commented Aug 27, 2026

Copy link
Copy Markdown
Owner

Overview

  1. This PR fixes part of [Feature Request]: Use new translation opportunity structure for all user-generated content in curated curriculum oppia/oppia#24933.
  2. This PR does the following:
    • Fixes Contributor Dashboard Selector Squeezing: Limits dropdown containers (.oppia-dropdown-row) to a clean, standardized 250px width on desktop and tablet views, aligning them to the right to prevent horizontal stretching and option truncation. Scales them to 100% width on mobile screens (max-width: 700px), stacking them vertically for readability and preventing cutoff.
    • Removes Category Translation: Removes the category field from the translatable contents in core/domain/exp_domain.py, keeping the lesson categories locked to system-defined names to avoid duplication in datastore filters.
    • Fixes Learner View Language Selector: Cleans up float and width conflicts on the lesson player start card (tutor-card.component.css and conversation-display.component.css), floating the language selector compactly to the top-right without overlapping.
    • Updates Puppeteer Acceptance Tests: Reduces the skip counts from 3 to 2 in translation Puppeteer specs (filter-the-translations, review-the-translations, translate-exploration-in-target-language, and check-their-accomplishment) to align with the removal of Category translation.
    • Updates Backend Unit Tests: Fixes the expected content counts (reduced by 1) in translation_opportunity_backfill_jobs_test.py, contributor_dashboard_test.py, and suggestion_test.py to match the Category translation removal.

Essential Checklist

Please follow the instructions for making a code change.

Testing doc (for PRs with Beam jobs that modify production server data)

N/A

Proof that changes are correct

Proof of changes on desktop with slow/throttled network

  • New Demo Video after Feedback (includes progress bar real-time updates and layout fixes):
newdemoafterfeedback.mp4
  • Desktop Layout Verification:
    • The dashboard dropdown filters display without being cut off or stretched horizontally, aligning neatly to the right.
    • The learner view language selector floats correctly to the top-right of the tutor card on a single line.

Verification via Automated Tests:

  1. Ran translation_opportunity_backfill_jobs_test to verify Beam audit job logic under the updated metadata counts:
    SUCCESS   core.jobs.batch_jobs.translation_opportunity_backfill_jobs_test: 32 tests (121.9 secs)
    
  2. Ran contributor_dashboard_test to verify translatable contents length assertions:
    SUCCESS   core.controllers.contributor_dashboard_test: 92 tests (89.1 secs)
    
  3. Ran suggestion_test to verify reviewable opportunities target counts:
    SUCCESS   core.controllers.suggestion_test: 83 tests (168.5 secs)
    

Proof of changes on mobile phone

N/A (covered in above video)

Proof of changes in Arabic language

N/A

PR Pointers

@github-actions

Copy link
Copy Markdown

Hi @Rohan-Unbeg, it looks like the description of this PR doesn't use our template, which is pre-populated when you open a new PR. Specifically, it's missing the following required sections:

  • Overview
  • Essential Checklist
  • Proof that changes are correct
  • PR PointersPlease re-open this PR and fill in the template without deleting any required sections.

@github-actions github-actions Bot closed this Aug 27, 2026
@Rohan-Unbeg Rohan-Unbeg reopened this Aug 27, 2026
arnav25102004 and others added 27 commits August 28, 2026 08:43
… components (oppia#27174)

* Fix part of oppia#27000: Add missing styleUrls to MultipleChoiceInput and ItemSelectionInput

* Fix part of oppia#26889: Add missing styleUrls to 17 components

* Fix part of oppia#26889: Add missing styleUrls to collection-player-page-root component
…uage … (oppia#27204)

Fix oppia#27040: Use native language text and lang attribute for language menu aria-label

Co-authored-by: NITISH KUMAR <imnitishkumar04@gmail.com>
…pia#27149)

* Removed all references

* Retaining check_tests_are_captured_in_ci.py

* Removing unused references

* Update yarn.lock

* restoring protractor config file

* fixing coverage

* Revert "Resolving conflict"

This reverts commit 082c95e, reversing
changes made to 82789d8.

* Revert "Revert "Resolving conflict""

This reverts commit 6de3f41.
…TML templates removing inconsistency of next 20 files (part-12) (oppia#27137)

Fix part of oppia#25941: Clean up legacy embedded <style> tags from HTML templates (part-12)
… to the new Learner Dashboard and support viewing submitted suggestions, replies,status updates. (oppia#27152)

* added backend api and upadted api responses according to the CUJs

* fixed flakes

* fixed flakes

* added main files for the features

* added top-navigation related logic

* removed top-profile navigation logic as things are much complicated and we need proper implementation doc for it

* fixed few review comments

* removed conflicts

* added unit tests andfixed flakes

* added apis for notifications

* added unit tests andfixed flakes

* added html file for creator side

* fixed mis-matching names

* added more unit tests and fixed flakes

* fixed a bug of showing notification on all three panels

* added more unit tests and fixed flakes

* fixed flakes

* added i18n tags

* added i18n tags

* added i18n tags

* added few fixes for updates when status is fixed

* added validation and exception for invalid status

* fixed resolve conflict

* fixed resolve conflict

* fixed failing frontend CI tests
* Upgrade @lhci/cli to 0.15.1 and lighthouse to 12.6.1

* Fix inject_entities_into_url to support URLs with multiple placeholders

* Remove unused FirebaseAdmin initialization from lighthouse setup script

* Fix LH tests

* Consolidate Lighthouse CI into a single prod-mode run

* Run Lighthouse CI for the desktop form factor

* Install Node dependencies under the shared Node 20 runtime

* Fix frontend tests and lint checks

* Add Chrome stability flags for lighthouse CI

* Relax lighthouse assertions to current per-page scores

* Fix lighthouse CI: relax donate assertions, reduce shard size, fix offscreen-images override

* Update lighthouse shard tests for 8 pages per shard

* Close stale pages between Lighthouse runs to prevent Target.getTargetInfo errors

* Relax accessibility thresholds to current per-page CI scores

* Fix donate page crash and revert to 12 pages/shard

Root cause: The puppeteer login script creates a new page via
browser.newPage() for every URL but never closes it. After 12 URLs,
12 stale login pages accumulate with active CDP sessions. When
Lighthouse connects and sets Target.setAutoAttach, it encounters
stale targets from prior runs and fails with Target.getTargetInfo:
Not allowed on the /donate page (and any subsequent page).

Fix: Close the login page after seeding blank pages for Lighthouse.
Login state (cookies) persists at the browser context level, not
page level, so closing the page is safe.

Also:
- Revert LIGHTHOUSE_PAGES_PER_SHARD from 8 back to 12
- Revert stale page cleanup in seedBlankPagesForLighthouseRuns
  (was ineffective since it only closed Puppeteer-session pages,
  not Lighthouse child process pages)
- Add accessibility thresholds for community-library (0.91),
  contact (0.95), contributor-dashboard (0.95), creator-dashboard
  (0.88)

* Fix LH assertion values

* Fix donate page CDP error and volunteer assertion

* Patch Lighthouse target-manager to handle cross-origin CDP errors

* Relax Lighthouse assertions for shard 1

* Relax Lighthouse accessibility assertions for shard 1

* Beautify Lighthouse CI logs with form factor headers

* Separate desktop and mobile Lighthouse assertion matrices

* Set ideal Lighthouse performance thresholds with per-form-factor values

* Always print LHCI stderr so assertion results are visible

* Add per-page error-level performance thresholds for shard 1

* Add per-page error-level performance thresholds for shard 2

* Add per-page error-level performance thresholds for shard 3

* Fix shard 1 mobile thresholds: add ~25% buffer for TBT and CLS\n\nMetrics fluctuate between CI runs. Per-page thresholds were set too\nclose to the worst observed values. New thresholds use ~25% buffer:\n- CLS: donate 0.1→1.0, contact 0.1→0.74, contributor-dashboard 0.1→0.15,\n  get-started 0.1→0.65\n- LCP: about 37900→42000, creator-guidelines 34300→38500\n- TBT: donate 1650→3200, about 510→830, admin 720→1210, blog-dashboard\n  860→1390, community-library 870→1380, contact 590→975, contributor-\n  dashboard 800→1290, creator-dashboard 950→1450, creator-guidelines\n  580→975, delete-account 750→1190, get-started 630→960

* Fix shard 2 mobile thresholds: add ~25% buffer for LCP and TBT\n\n- moderator: lcp 34500→38500, tbt 1240→1750\n- preferences: tbt 1360→1950\n- teach: tbt 870→1210\n- topics-and-skills-dashboard: lcp 36500→40500, tbt 1260→1730\n- thanks: tbt 710→920\n- volunteer: tbt 930→1170

* Bump admin and community-library LCP thresholds

* Fix privacy-policy LCP, profile TBT, volunteer CLS

* Promote catch-all to error level, fix desktop thresholds

* Fix catch-all SI/LCP, contributor-dashboard SI, desktop teach

* Fix linter errors and volunteer FCP/SI thresholds

* Fix donate LCP and about FCP/SI thresholds

* Scale all perf thresholds by 1.5x

* Remove stale TODO comments from lighthouse configs

* Fix review comments and consolidate accessibility mechanism

* Fix story page desktop accessibility threshold

* Fix story page desktop accessibility threshold

* Filter warn-level Lighthouse results from CI

* Add lighthouse env and patch tests

* Harden lighthouse node patch and heap

* Reuse playwright node for lighthouse install

* Remove stray pylint directives from lighthouserc

* Resolve duplicate patch tests

* Fix lighthouse test patch

* Fix lighthouse test expectations
…lModel records (oppia#27041)

* Prevent failed emails from creating SentEmailModel records

* Made suggested changes

* Fixed Lint

* Revert type hint to str
…TML templates removing inconsistency of next 20 files (part-11) (oppia#27120)

Fix part of oppia#25941: Clean up legacy embedded <style> tags from HTML templates (part-11)
* Fix 16309 remove webpack infra

* Enforce strict TypeScript checks and finalize test configurations

* Updated comment in karma and combine tests

* Fixed Frontend coverage test and update to code-coverage test command for Angular CLI

* Fix oppia#15185: Add translation progress bar to review tab (oppia#26730)

* Fix oppia#15185: Add translation progress bar to review tab

Wires the existing progress-bar props from the translate tab
(added in oppia#15182) into the reviewable translations opportunity
list, so reviewers can see per-lesson translation status
(cards accepted / in review / available) before opening a
lesson to review.

* Address review: keep review-tab button enabled when complete

The Review Translations tab reuses the same opportunities-list-item
progress bar as the Translate Text tab, which was disabling the
opportunity button once translationsCount + inReviewCount reached
totalCount. On the translate tab that's correct (nothing left to
translate), but on the review tab it hid the ability to open a
lesson that still has cards pending review.

Add a disableButtonOnComplete input (default true, so the translate
tab is unaffected) and set it to false for the review tab.

* Retrigger CI after transient pip bootstrap failure

The previous CI run failed across all jobs (Backend, Lint, Type,
Frontend, Full-stack) at the same environment-setup step with:
ImportError: cannot import name 'BuildDependencyInstallError' from
'pip._internal.exceptions'. This is a transient GitHub Actions
runner/pip issue unrelated to this PR's changes (confirmed by a
successful develop-branch Backend unit tests run at the same time).

* Retrigger CI: re-run flaky acceptance test (unrelated to PR changes)

---------

Co-authored-by: Rohan Unbeg <rohanunbeg0918@gmail.com>
Co-authored-by: NITISH KUMAR <imnitishkumar04@gmail.com>

* Fix oppia#24938: Migrate remaining userJourneys.js tests to acceptance  (oppia#25268)

* prettier fix

* linter fix

* lo.13 tests

* lc.1 tests

* IP1 remainder testing

* error resolution

* error fixing

* as previous

* debugging

* error fixing for card selections

* finalizing fixes for exploration

* logged out learner tests

* formatting fixes

* formatting

* minor fixes

* reverting tsconfig issue

* reverting tsconfig issue

* reverting git ignore

* test

* fixing req file

* .

* fix

* fix

* fix

* removing userjourneys.js

* removing navigationPromise approach

* fixing minor test failure

* resolving error

* fixing failing mobile test

* linter fixes

* linter fixes

* linter fixes

* verifying push

* push test

* resolving error

* linter fix

* time difference check

* error resolution

* fix

* error fix

* linter fix

* error resolution

* debugging

* resolving dev mode failures

* linter fixes

* merge conflict res

* reverting file

* linter fix

* merge conflict resolve

* restoring root-files-config

* adding missing file to rootfilesconfig to resolve ci error

* fix

* review fixes

* resolving discrepencacies with sheet

* error resolving

* resolving errors after fixing discrepency

* view collection error fix

* learn about org fix

* adding spreadsheet link, update year

* mobile fix

* same collection card for mobile error

* view collection error fix selector

* scroll fix

* adjust footer wait

* exploration creation mobile error fix (uncovering learner area bc of dropdown overlay

* prev error fix

* making constant for selectors

* date update

* date update

* error fix overlay block

* verifying fix

* fixing links

* link update

* creating functions to ensure spec is high-level only

* adding selector constants, removing unnecesarry waits

* removing unnecesarry wait

* added missing descs for card, removed unnecesarry selector

* adding constant for selector

* replacing unnecesarry tries w direct dom click

* reduced role check time

* removed unnecesarry wait

* making view-a-collection spec high level, shifting funcs to loggedoutuser ts file

* edit

* making create basic exp higher level

* removing duplicate selectors

* fixing failing test (ts error, moving func)

* small fix

* error fix

* test

* test

* linter fix

* adding framework label

* reverting change

* removing unnecesarry change

* as prev

* creating func for lowlevel code

* indent fix

* lint fix

* syntax error fix

* minor fix

* linter fix

* matching spreadsheet

* bold and italic text check

* fixing accidental change

* lint fix

* removing unused files (ModeratorPage, PreferencePage)

* fix

* wait fix

* test

* fix

* removing unused vars

* test

* fix

* fix

* fix

* fix

* fix

* fix

* fix test

* removed comments

* removed set viewport call and collection_name constant

* new getTextContent func

* reversing change

* removing retry, using selector

* fix test

* linter fix

* reverting change

* linter fix

* removing retry testing in super admin

* as prev

* as prev

* test

* reverting change

* adding params

* testing fix

* testing func removal

* removing unnecesarry func as suggested

* view fix

* edit

* fix

* test

* revert failing fix

* conditional fix

* reverting fix

* test

* testing storage clear

* small fix

* fix test

* fix test

* revert

* super admin removal

* removing storage clear

* removing unneeded role

* skipping mobile test

* linter fix

* linter fix

* linter fix

* Removed unnecessary changes

* Fixing lint

* Resolved conflicts

* Fixing failing tests

---------

Co-authored-by: Mohak <143473709+Mohak51234@users.noreply.github.com>

* Fix oppia#18670: Fix o.getOpportunitySubheading is not a function (oppia#26935)

refactor opportunity iteration

Co-authored-by: NITISH KUMAR <imnitishkumar04@gmail.com>

* Remove unsafe type check and fix test infra

* Revised memory size and reverted captureKilltime

* Fixed backend failing error

* Revert ck-editor and added comment

* Fixed Frontend error

* updated capturekilltime and fixed frontend coverage

* Add ts-nocheck for spec.ts files

* Fixed failing frontend and lint check

* Reverted check_frontend_test_coverage to original

* Fixed failing lint throw error

* Fixed backend coverage

* Fixed frontend test

---------

Co-authored-by: Bhuvan Somisetty <somisettybhuvan5@gmail.com>
Co-authored-by: Rohan Unbeg <rohanunbeg0918@gmail.com>
Co-authored-by: NITISH KUMAR <imnitishkumar04@gmail.com>
Co-authored-by: haani-maybe <171415359+haani-maybe@users.noreply.github.com>
Co-authored-by: Mohak <143473709+Mohak51234@users.noreply.github.com>
Co-authored-by: Sheron Jayasinghe <192441684+sheronjay@users.noreply.github.com>
* Fix added translation and question coordinators tabs

* Added coverage for the contribution-admin-stats-table

* removed unwanted file

* Updated file location

* Updated file location

* Fixed the typeerror after changing the location of file
…TML templates removing inconsistency of next 30 files (part-13) (oppia#27163)

* Fix part of oppia#25941: Clean up legacy embedded <style> tags from HTML templates removing inconsistency of next 30 files (part-13)

* added skipped file

* Fixed backend lint error

* updated allowlist

* restored correct html style allowlist

* Fixed lint error
…hotToMatch (oppia#27209)

* Updated the message

* Update snapshot

* Added link to wiki

* Updating error message

* Added custom reporter to prevent attachment logs

* Updating msg

* Correcting hyperlink and removing indentation
…TML templates removing inconsistency of next 20 files (part-6) (oppia#26932)

* Fix part of oppia#25941: Clean up legacy embedded <style> tags from HTML templates (part-6)

* Remove 20 extracted CSS files from html style tag allowlist

* restored fixed .txt allowist file as merge conflict

* fixed allowlist .txt file

---------

Co-authored-by: Kartik Suryavanshi <158498247+KartikSuryavanshi@users.noreply.github.com>
Co-authored-by: Hardik Goyal <135348086+HardikGoyal2003@users.noreply.github.com>
* Fix oppia#27236: Fix flaky background layout in addTranslationRightsModal by waiting for language fetch

* Update baseline screenshots for stable translation modal state

* Remove unnecessary wait condition from third test

* Wait for stats table and update baseline

* Fix wait condition to look for no-data message

* Fix mobile baseline filename

* Fix scroll position race condition on mobile

* Address review comments: Extract selector constants
… UI issues (oppia#27200)

* Fixed css and ui as per demo

* added amy's suggestion

* fixed practice button

* fixed breadcrumb

* addressed coderabbit's comment

* fixed checks and issues

* fixed comment

* Removed unrelated code, inconsistencies in the code

* fixed failing tests

* fixed merge conflicts

* fixed navbar

* added translations back

* added translations back

* fixed lint checks and frontend tests

* updated the menu-book-icon

* fixed checks

* Added more tests for coverage

* added padding as suggestions from katrina

* fixed all tests

* fixed acceptance tests

* fixed acceptance tests failure

* fixed the navbar

* finalised ui

* added tests for coverage

* fixed mastery challenge

* added practice icon and increased lesosn icon size

* fixed practice color in navbar

* added tests

* implemented the new navbar

* expanded lesson view

* removed completed badge

* fixed changes requested from Amy found during testing

* fixed frontend test

* added tests for coverage

* renamed adventure --> Module

* fixed css

* fixed lint

* fixed acceptance tests

* renamed view mastery to unlock mastery

* addressed coderabbit's comments

* Renamed Adventure --> Module

* Added story-section business logic to service file

* fixed lint check

* addressed review comments
…or LoggedOut User journey of Web User Feedback project(LO.15, LO.16). (oppia#27133)

* added files and ui changes

* added files

* fixed broken ci and en

* fixed broken ci and en

* made changes according to new CUJs

* added LO.15 suite

* made changes according to new CUJs

* added LO.15 suite

* added LO.15 suite

* Added screenshots for dev mobile

* did changes as requested by reviewers

* added changes related to review comments

* added screenshots

* added missing screenshots

* added screenshots

* fixed flaky behaviour

* fixed flaky behaviour

* removed unused utilities
* Add formula text warnings

* Fixed Typescript tests

* Fixed frontend tests

* Fixed lint

* Fixed full stack tests

* Addressed review comments

* Added coverage

* Add minimize and maximize toggle to math formula warning

* Fixed lint

* Addressed review

* Fixed line breaks

* made suggested changes

* Testing

* Made suggested changes

* Removed duplicate test

* Fixed Lint

* Fix type checks

* Fix frontend tests

* Fix Tests

* Fixed more tests

* Address Review comment

* Fixed Lint

* Fix Subscription typescript error

* Fix typescript error

* Fix i18n

* Address review

* Fixed Lint

* Re-add math-formula-detection.service.ts in CODEOWNERS
…ssion Exit Alert Handling. (oppia#27273)

* Remove stubbed assessment interrupt feature and warn before leaving active attempt

The interrupt card could never be shown: showAssessmentInterruptCard was
only ever set to false, so the component and its wiring were dead code.
Remove the component, its module registration, the state flag and handlers,
and the related i18n keys.

Also register a beforeunload guard while an attempt is in progress so
learners are warned before closing or refreshing the page and potentially
losing unsaved answers. The guard is removed once answers are submitted or
the component is destroyed.

* Add exit confirmation to certificate assessment question player

* Remove stale I18N_ASSESSMENT_INTERRUPT keys from remaining locale files

* Rename isAttemptSubmitted to attemptIsSubmitted and add exit comment

---------

Co-authored-by: mosin74 <mosin74@gmail.com>
…ions tab (oppia#27251)

Fix part of oppia#18510: Deprecate voiceover recording in the editor translations tab
* Consolidate Lighthouse CI configuration

* Add lighthouse route coverage linter

* Fix lint and type errors in linter

* Fix lint issues in lighthouse route linter

* Fix multiline key test mock

* Update exclusion list comment

* Parse route objects with brace depth

* Harden lighthouse node patch and heap

* Reuse playwright node for lighthouse install

* Remove stray pylint directives from lighthouserc

* Rename admin page constant

* Fix merge conflict files
* Fix analytics script

* Fix frontend tests

* Fix N/A: Add descriptive failure messages to Playwright expectScreenshotToMatch (oppia#27209)

* Updated the message

* Update snapshot

* Added link to wiki

* Updating error message

* Added custom reporter to prevent attachment logs

* Updating msg

* Correcting hyperlink and removing indentation

* Fix part of oppia#25941: Clean up legacy embedded <style> tags from HTML templates removing inconsistency of next 20 files (part-6) (oppia#26932)

* Fix part of oppia#25941: Clean up legacy embedded <style> tags from HTML templates (part-6)

* Remove 20 extracted CSS files from html style tag allowlist

* restored fixed .txt allowist file as merge conflict

* fixed allowlist .txt file

---------

Co-authored-by: Kartik Suryavanshi <158498247+KartikSuryavanshi@users.noreply.github.com>
Co-authored-by: Hardik Goyal <135348086+HardikGoyal2003@users.noreply.github.com>

* Fix flaky translation modal 27236 (oppia#27302)

* Fix oppia#27236: Fix flaky background layout in addTranslationRightsModal by waiting for language fetch

* Update baseline screenshots for stable translation modal state

* Remove unnecessary wait condition from third test

* Wait for stats table and update baseline

* Fix wait condition to look for no-data message

* Fix mobile baseline filename

* Fix scroll position race condition on mobile

* Address review comments: Extract selector constants

* [GSoC 2026] M2.8 - Fix part of oppia#19614: Address demo feedback and UI issues (oppia#27200)

* Fixed css and ui as per demo

* added amy's suggestion

* fixed practice button

* fixed breadcrumb

* addressed coderabbit's comment

* fixed checks and issues

* fixed comment

* Removed unrelated code, inconsistencies in the code

* fixed failing tests

* fixed merge conflicts

* fixed navbar

* added translations back

* added translations back

* fixed lint checks and frontend tests

* updated the menu-book-icon

* fixed checks

* Added more tests for coverage

* added padding as suggestions from katrina

* fixed all tests

* fixed acceptance tests

* fixed acceptance tests failure

* fixed the navbar

* finalised ui

* added tests for coverage

* fixed mastery challenge

* added practice icon and increased lesosn icon size

* fixed practice color in navbar

* added tests

* implemented the new navbar

* expanded lesson view

* removed completed badge

* fixed changes requested from Amy found during testing

* fixed frontend test

* added tests for coverage

* renamed adventure --> Module

* fixed css

* fixed lint

* fixed acceptance tests

* renamed view mastery to unlock mastery

* addressed coderabbit's comments

* Renamed Adventure --> Module

* Added story-section business logic to service file

* fixed lint check

* addressed review comments

* [GSoC 2026] M2.9.1 - Fix part of oppia#24716: Added Acceptance test for LoggedOut User journey of Web User Feedback project(LO.15, LO.16). (oppia#27133)

* added files and ui changes

* added files

* fixed broken ci and en

* fixed broken ci and en

* made changes according to new CUJs

* added LO.15 suite

* made changes according to new CUJs

* added LO.15 suite

* added LO.15 suite

* Added screenshots for dev mobile

* did changes as requested by reviewers

* added changes related to review comments

* added screenshots

* added missing screenshots

* added screenshots

* fixed flaky behaviour

* fixed flaky behaviour

* removed unused utilities

* Fix oppia#20239: Add formula text warnings (oppia#26726)

* Add formula text warnings

* Fixed Typescript tests

* Fixed frontend tests

* Fixed lint

* Fixed full stack tests

* Addressed review comments

* Added coverage

* Add minimize and maximize toggle to math formula warning

* Fixed lint

* Addressed review

* Fixed line breaks

* made suggested changes

* Testing

* Made suggested changes

* Removed duplicate test

* Fixed Lint

* Fix type checks

* Fix frontend tests

* Fix Tests

* Fixed more tests

* Address Review comment

* Fixed Lint

* Fix Subscription typescript error

* Fix typescript error

* Fix i18n

* Address review

* Fixed Lint

* Re-add math-formula-detection.service.ts in CODEOWNERS

---------

Co-authored-by: Mohak51234 <143473709+Mohak51234@users.noreply.github.com>
Co-authored-by: Aditya Lakhotia <adityalakhotia21@gmail.com>
Co-authored-by: Kartik Suryavanshi <158498247+KartikSuryavanshi@users.noreply.github.com>
Co-authored-by: Hamza Anwar <anwarhamza1408@gmail.com>
Co-authored-by: NITISH KUMAR <imnitishkumar04@gmail.com>
Co-authored-by: Puneet Seervi <puneet9591@gmail.com>
… once instead of making a request for each next question (oppia#27256)

* Return all question state in the start handler

* Fix type errors in certificate assessment player spec tests

Add the required question_state_data field to mock question dicts and
replace the non-existent toBeTrue/toBeFalse matchers with toBe(true)/
toBe(false) in the certificate assessment player page tests.

* Batch question-snapshot lookups and rename pinning test

* Fixed mypy error

* Lint checks

* Inline loadQ1 alias to load in spec

* add alerts, internet connectivity and translate service mocks to spec

* Merge branch 'develop' into eager-question-loading

---------

Co-authored-by: mosin74 <mosin74@gmail.com>
…out, category translation, and learner view styling
…cceptance tests for Category translation removal
@Rohan-Unbeg
Rohan-Unbeg force-pushed the gsoc-m2-feedback-fixes branch from b28646e to 828b42e Compare September 10, 2026 17:10
@github-actions

Copy link
Copy Markdown

Hi @Rohan-Unbeg, this PR's branch appears to have been force-pushed or rebased, which rewrites commit history. Per our contribution guidelines, force-pushing to a PR branch is not allowed, so this PR is being closed. Please open a new PR from a fresh branch instead of force-pushing. Thanks!

@github-actions github-actions Bot closed this Sep 10, 2026
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.