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

Add/google calendar #236

Draft
wants to merge 15 commits into
base: develop
Choose a base branch
from
Draft

Add/google calendar #236

wants to merge 15 commits into from

Conversation

MistahSanta
Copy link
Contributor

Integrated Google Calendar with Events.

This is a draft PR to get some feedback and ideas to add more to

Added two buttons:

  • One that is connected with Google Calendar API whenever the user sign in with google
    image
    Right now, added text "auth" to be able to differentiate between the two button. Before this PR is merge, will remove that auth text
  • Another button that is not connected with Google Calendar API, but will display a pop up that allow the user to add events to Google Calendar or Outlook Calendar via .ics
    image

Future Implementation:

  • Figure out a way to only ask user for google Calendar permission when user actually click on "add to calendar" button rather than requiring user to give us access to google Calendar right when user sign in with google
  • Perhaps figure a way to implement Apple Calendar with it
  • Right now, I think Microsoft Outlook Calendar does not work
  • Verify Google Calendar Account because every time we ask user for more permission, user is directed to this page first:
    image

Tested:

  • Able to add events using Google calendar ( with API and without API ):
    image
  • Refresh_token rotation seem to work ( but done limited tested as of this moment - only waited 3 hours)

Copy link

vercel bot commented Sep 2, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
jupiter ❌ Failed (Inspect) Sep 28, 2024 5:19am

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