Skip to content

M2.6 2#85

Closed
jayam04 wants to merge 117 commits intodevelopfrom
m2.6-2
Closed

M2.6 2#85
jayam04 wants to merge 117 commits intodevelopfrom
m2.6-2

Conversation

@jayam04
Copy link
Copy Markdown
Owner

@jayam04 jayam04 commented Sep 26, 2025

Overview

  1. This PR fixes or fixes part of #[fill_in_number_here].
  2. This PR does the following: [Explain here what your PR does and why]
  3. (For bug-fixing PRs only) The original bug occurred because: [Explain what
    the cause of the bug was, and which PR introduced it]

Essential Checklist

Please follow the instructions for making a code change.

  • I have linked the issue that this PR fixes in the "Development" section of the sidebar.
  • I have checked the "Files Changed" tab and confirmed that the changes are what I want to make.
  • I have written tests for my code.
  • The PR title starts with "Fix #bugnum: " or "Fix part of #bugnum: ...", followed by a short, clear summary of the changes.
  • I have assigned the correct reviewers to this PR (or will leave a comment with the phrase "@{{reviewer_username}} PTAL" if I can't assign them directly).

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

  • A testing doc has been written: ... (ADD LINK) ...
  • (To be confirmed by the server admin) All jobs in this PR have been verified on a live server, and the PR is safe to merge.

Proof that changes are correct

Proof of changes on desktop with slow/throttled network

Proof of changes on mobile phone

Proof of changes in Arabic language

PR Pointers

diff --git a/core/templates/pages/admin-page/admin-page.module.ts
b/core/templates/pages/admin-page/admin-page.module.ts index
c52b4abf47..1fcdccd46c 100644 ---
a/core/templates/pages/admin-page/admin-page.module.ts +++
b/core/templates/pages/admin-page/admin-page.module.ts
The commit removes redundant tab switching code since the functionality
was already handled by changeTabInInteractionSelectionModal method.
@github-actions
Copy link
Copy Markdown

🛠️ Hi @jayam04, it looks like the full-stack tests are failing on your PR. Please check the logs and follow these instructions to fix the issues, or report the error as a flake. Thanks!

@testbot-oppiabot
Copy link
Copy Markdown

Hi @jayam04, I'm going to mark this PR as stale because it hasn't had any updates for 7 days. If no further activity occurs within 4 days, it will be automatically closed so that others can take up the issue.
If you are still working on this PR, please make a follow-up commit within 4 days (and submit it for review, if applicable). Please also let us know if you are stuck so we can help you!

@testbot-oppiabot
Copy link
Copy Markdown

Hi @jayam04. Due to recent changes in the "develop" branch, this PR now has a merge conflict. Please follow this link if you need help resolving the conflict, so that the PR can be merged. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant