Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Loading Page Documentation #247

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

shah-esha
Copy link
Collaborator

Summary

This pull request adds documentation to the LoadingPageWidget class, explaining its functionality, parameters, and behavior.

  • Added documentation for LoadingPageWidget class

@dti-github-bot
Copy link
Member

[diff-counting] Significant lines: 45.

@3TTemi
Copy link
Contributor

3TTemi commented Mar 3, 2025

Documentation looks good! Remember to delete the test comments so that they are not pushed into master. Also i'm a bit confused on the if condition changes made in journeys_page.dart, was that also a part of something that was trying to be accomplished in this PR?

3TTemi and others added 11 commits March 2, 2025 23:44
* Ranking List Names Updated with Ellipsis

* truncate with flutter ellipsis

* Removed Custom Function repalced with flutter ellipsis
* Added state to disable button after click

* Add documentation to GameplayMap Widget
* Fix Future Async conflict casuing app freeze

* Transition to MediaQuery for home page and challenge cells

* Implemented custom padding adjustments for Iphone and Android

* Add documentation for navigation between home, search bar/filtering, and challenges

* Add constant device varibales instead of MediaQuery Calls
* Fixed local machine dependency issues

* Fixed the issue of registering new users on simulator

* Added documentation for login related pages

* Fixed the wrong uri call for checking user existence

* Removeed loggings that are used for debugging

* Made check user existence logic concise
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.

4 participants