Skip to content

Comments

Ac 3 Added migration files, notifications, minor changes#1465

Merged
ChaudharyRaman merged 15 commits intoFusionIIIT:ac-3from
Daksh119:ac-3
Apr 17, 2024
Merged

Ac 3 Added migration files, notifications, minor changes#1465
ChaudharyRaman merged 15 commits intoFusionIIIT:ac-3from
Daksh119:ac-3

Conversation

@Daksh119
Copy link

@Daksh119 Daksh119 commented Apr 15, 2024

Issue that this pull request solves

Closes: # (issue number)

Proposed changes

Added migration files, notifications and some UI improvements

Types of changes

Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other (please describe):

Checklist

Put an x in 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 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 does not break the current system and it passes all the current test cases.

Video

https://drive.google.com/file/d/1-P5nUdyaw1OFBKrjV88aNoFrtuV-MJfm/view?usp=sharing

Other information

Any other information that is important to this pull request

@Priyansh61
Copy link

@Daksh119 kindly complete the PR template, wont be able to review without that

@Daksh119
Copy link
Author

Sure

@Daksh119
Copy link
Author

@ChaudharyRaman

@ChaudharyRaman
Copy link
Contributor

@Daksh119 can you remove all the migrations manually and push ??

@Daksh119
Copy link
Author

@ChaudharyRaman I can but I added them only because we were instructed to do so.

@Priyansh61
Copy link

@ChaudharyRaman PTAL

@ChaudharyRaman
Copy link
Contributor

LGTM Merging !!!

@ChaudharyRaman ChaudharyRaman merged commit a8f8069 into FusionIIIT:ac-3 Apr 17, 2024
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.

7 participants