social-platform-donut-frontend
social-platform-donut-frontend copied to clipboard
Google calendar integration
#349
Initial setup for the frontend portion of the Google Calendar integration. Currently, the calendar is displayed on a separate page and is accessed by clicking on the "My Calendar" button on the dashboard.
However, as discussed with @jaskirat2000 and @devesh-verma There's a lot to be changed from the current implementation mainly the way the integration is accessed and how the integration is presented (i.e a popup instead of separate page). These changes will be made in the coming days.
Deploy preview for donut-frontend-r ready!
Built with commit 8eba070c90b4fabbe4c50a59f0f060f6c714bddd
https://deploy-preview-350--donut-frontend-r.netlify.com
Initial Setup is good, I am looking forward to a new UI as discussed on the Slack. ;) @devesh-verma @vaibhavdaren @Rupeshiya Your thoughts on this?
SPA--> no page change
@AuraOfDivinity hey can you please solve merge conflict.
@devesh-verma I have fixed the merge conflicts. (Removed the button we had in the news feed since we will be accessing the integration from the sidebar).
I'm currently working on the changes requested by @jaskirat2000.
@AuraOfDivinity cool
@jaskirat2000 @devesh-verma @vaibhavdaren I have made changes to the frontend as suggested by @jaskirat2000 . The current iteration can be seen in the following image.
Few issues I'm having at the moment,
-
The token obtained from the backend potion at https://github.com/codeuino/social-platform-donut-backend/pull/63 is currently stored in the local storage and is not refreshed meaning that the authentication will be required after 15 minutes of use.
-
Events can only be added at the moment and have to be deleted separately from the google calendar itself.
-
The add event pop up is not validated.
-
This calendar is currently accessed through /calendar route. This should be fixed in time when the integrated apps page and the integrated apps part of the sidebar is completed.
The rest of the integration seems to be working fine in my opinion. I would love to hear your thoughts/suggestions and any improvements you would like to see both in terms of frontend and the backend parts.
Thanks. I am wondering that will the user have to login again after every 15 mins?
Also where is the integration button located specifically for Google Calender?
Do you know about refresh tokens?
@AuraOfDivinity please squash your commits.
use git pull --rebase origin developement
from next time.
@jaskirat2000 @devesh-verma @vaibhavdaren Apologies on the delay. I have made requested changes to calendar integration.
-
The auth token is no longer passed to the frontend as it previously did. As @vaibhavdaren mentioned I'm now resorting to saving the auth token on our database. This means we no longer have to expose the auth token to the clients.
-
Unlike in the previous iteration, the auth token is refreshed automatically when expired using the refresh token saved in the database. This means no additional logins are required after the initial login.
-
The add event modal is now validated and displays a set of errors accordingly. (Check the image below).
-
The calendar is currently accessed through the /calendar route. Once we have completed the app integrations page and the sidebar tray potion of the integrated apps, I'll work on adding the calendar to the required positions.
I have also made the necessary changes to the backend as well https://github.com/codeuino/social-platform-donut-backend/pull/63
Do kindly give me your feedback on this!
Nice work!! 👍 well done !! 🥇
@devesh-verma lets merge this.
@AuraOfDivinity Please squash the commits into one. You are good to go. Great work, really appreciable!
@AuraOfDivinity Please resolve the conflicts.