Merging AC-final( AC-1,AC-2, AC-3,AC-7) into latest_staging #1489
Merged
prabhatsuman merged 124 commits intolatest_stagingfrom Apr 22, 2024
Merged
Merging AC-final( AC-1,AC-2, AC-3,AC-7) into latest_staging #1489prabhatsuman merged 124 commits intolatest_stagingfrom
prabhatsuman merged 124 commits intolatest_stagingfrom
Conversation
…tructor (#1075) Co-authored-by: “tech-akash” <“20bcs020@iiitdmj.ac.in”>
* implemented authentication and sidenavbar * added announcement section
* JY , 16/02/2024 6:26PM * JY 17/02/24 11:15 AM Implemented hidden_grades * Added apis and models --------- Co-authored-by: Jatin Yadav <jatinyadav@Jatins-MacBook-Pro.local>
…ated html templates and apis for the same
Merge AC-7 to AC-Final
dashboard affected by AC-7 corrected
Author
|
@ChaudharyRaman @prajjwalkapoor @Priyansh61 Verify this PR. |
added __init__.py and updated 0000_initial.py for AC-7, Please merge this in ac-final before merging ac-final to latest_staging
Ac 3 minor change
Ac 3 fix errors related to grades for examination module
removed duplicate Media_Root
Ac 3 added version field in urls and fixed course fetching errors
AC-3 to AC-final
prabhatsuman
changed the title
Author
|
@ChaudharyRaman LGTM. u can proceed to merge |
Added AC-2 Module Required APIs
Ac-1 API changes to AC-final
Author
|
@ChaudharyRaman AC-7 has used the wrong tables in their code but their Pull request cannot be reverted so we are deploying ac-final with ac-7. If they correct the code, we will merge it again to the latest staging. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
17 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue that this pull request solves
Closes: # (issue number)
Proposed changes
The changes made in this pull request ensure that AC-1, AC-2, AC-3 and AC-7 are working properly.
Types of changes
Put an
xin the boxes that apply:Based on the information provided, it sounds like these changes may be a bug fix or other (testing and maintenance). Choose the appropriate type of change for your pull request.
Checklist
Put an
xin the boxes that apply:My code follows the style guidelines of this project.
I have performed a self-review of my own code.
I have created a new branch for this pull request.
I have commented my code, particularly in hard-to-understand areas.
I have made corresponding changes to the documentation.
My changes generate no new warnings.
My changes do not break the current system and pass all the current test cases.
Add a checkmark for the items you have completed.
Screenshots
If applicable, please attach screenshots of the changes made, particularly for changes affecting the user interface.
Other information
The code was tested by merging the AC-final branch into the latest_staging branch of my forked repository. This testing process has confirmed that AC-1, AC-2, AC-3 and AC-7 are working properly.
If there is any other important information related to the pull request, such as known issues or further testing notes, you can include that here.
Remember to include the appropriate issue number in the "Closes" line and mark the appropriate types of changes and checklist items. Let me know if you have any more points to add or any other specific details.