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

Misc Changes #21

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Misc Changes #21

wants to merge 6 commits into from

Conversation

hk21702
Copy link

@hk21702 hk21702 commented Aug 16, 2022

Migrated most of the configuration to an INI file which should make it more friendly for users. Also did a lot of refactoring using an automated tool that should cut down on redundancies.

Changed the calendar token generation process to use a local server instead of client. This removes the need to copy paste in a token key yourself into the console. The client system also had some errors on my side on not recognizing scope.

Fixed a bug where it would look for organizers instead of creator when creating gCal_calendarId. Organizer is unrelated to the actual calendar itself, and if this were an autogenerated event, from say an email invite or from Google, the organizer would be from an email not registered in the dictionary resulting in an error.

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.

1 participant