Pranjal360Agarwal / ApnaBharat-Bus-Booking-Reservation-System

It is the system containing fully functional features from booking ticket to generating the ticket. With the bus ticket reservation system, you can manage/book reservations, clients data and passengers lists, can add new bus or new route as an Operator admin.
https://apna-bharat-bus-booking-reservation-system.vercel.app
MIT License
72 stars 100 forks source link

[GSSoC 23'] Successfully Added Dark Theme Button & Bug Resolve #117

Closed HartzFrequency closed 1 year ago

HartzFrequency commented 1 year ago

Hello, @Pranjal360Agarwal ! I've implemented a dark theme button for switching between tkinter themes. Additionally, I've fixed a bug and I'll provide more information about the changes in the following comment.

https://github.com/Pranjal360Agarwal/ApnaBharat-Bus-Booking-Reservation-System/issues/109 My Issue Number

HartzFrequency commented 1 year ago

Hi @Pranjal360Agarwal , here are the details of the changes made

BUG ENCOUNTERED

If you previously encountered a situation where you could switch from the sign-in window to the main window without entering the correct username and password, that issue has been resolved. Now, if you close the sign-in window without providing the necessary credentials, the program will halt and further execution will be prevented.

BUG is resolved using the function

def Transition_LoginToMain():
    with open('Modules\\SignIn_Module\\SignIn_Check.txt', 'r') as file:
        content = file.read()
    value = bool(content)

    if not value:
        exit()

    with open('Modules\\SignIn_Module\\SignIn_Check.txt', 'w') as file2:
        file2.truncate(0)

ABOUT THE THEME

image

image

image

Pranjal360Agarwal commented 1 year ago

great @HartzFrequency

HartzFrequency commented 1 year ago

hi @Pranjal360Agarwal, thanks for merging the PR, can you please assign labels to it, it will be so great of you! Thank you @Pranjal360Agarwal

gurjeetsinghvirdee commented 1 year ago

@Pranjal360Agarwal Kindly add gssoc23 & pointa label on merged PRs