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.

Sign up for GitHub

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

Jump to bottom

Revamp chrome extension #31

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

Revamp chrome extension #31

wants to merge 26 commits into from

Conversation

tanmoysrt
Copy link
Member

@tanmoysrt tanmoysrt commented Dec 30, 2022 •
edited
Loading

Referenced to the issue #30

Tasks

  • UI Design Design Here
  • Convert to manifest v3. As support of manifest v2 going to removed in 2023
  • OAuth 2.0 Implicit Token Based Authorization
  • Implementation of API for Insert Image and add hyperlink to it
  • UI Implement
  • Integrate UI

Improvements :

  • Upgrade to Manifest V3 (as chrome store will stop support for V2 from 2023)
  • Removed dependency on google slides add-on by implementing UI interface and logics in client side
  • New UI for Extension
  • Make the extension more user friendly

Proposed Design :

Login Screen Embed Tool Screen
Frame 19 Frame 20

Final Preview :

31_dec-2022-12-31_13.04.16.mp4

Note :
Client ID of oauth need to be updated in embedTool.js before merging

@tanmoysrt
update docs link in broser_action
04249f3
@tanmoysrt
upgraded to manifest V3
2ec9eb1
@tanmoysrt
added updated link of google slides embed guide
36ab27e
@tanmoysrt
remove unnecessary commented sample codes
870826e
@tanmoysrt
as manifest v3 has no support for chrome.extension and it inject code…
3e4321b 
… at document idle state only
@tanmoysrt
use onreadystatechanged listener in place of setInterval
7edb286
@tanmoysrt
renamed some functions
3231e6a
@tanmoysrt
initialization code rewritten
08eb2ee
@tanmoysrt
added gsi client library locally
f5f4b2b
@tanmoysrt
inject parts of embed tool through inject.js
63b1c19
@tanmoysrt
axios added for api calling
6abce4c
@tanmoysrt
add circuitverse.org as authorized domains
c8d5060
@tanmoysrt
add googleapis.com as authorized domains
37c28f7
@tanmoysrt
add slides.googleapis.com as authorized domains
eeb937b
@tanmoysrt
presentation loading page uri updated with presentations
b541ea7
@tanmoysrt
service-worker background script completed
990b55c
@tanmoysrt
working an expected ui update required
3de51b4
@tanmoysrt
img folder and required icons for design added
1b91c20
@tanmoysrt
new imgs added to web_accessible_resources
80b33b4
@tanmoysrt
fetch calls are async , so modified to ensure after html got inserted…
4bb329d 
… js insert
@tanmoysrt
removed loading.gif
1bc35d6
@tanmoysrt
implementation coomppleted
dce8c12
@tanmoysrt
update popup.html text
7ad10cf
@tanmoysrt
stylesheet update
dda3635
@tanmoysrt
version update
6922ae7
@tanmoysrt tanmoysrt marked this pull request as ready for review December 31, 2022 08:03
@tanmoysrt
Copy link
Member Author

@tachyons @vedant-jain03 can you take a look at this and provide some feedback. It's pending for more than a month.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tanmoysrt