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 firefox extension #33

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

Revamp firefox extension #33

wants to merge 2 commits into from

Conversation

tanmoysrt
Copy link
Member

@tanmoysrt tanmoysrt commented Jan 1, 2023 •
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 :

1_jan-2023-01-01_16.56.42.mp4

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

@tanmoysrt tanmoysrt marked this pull request as ready for review January 1, 2023 11:30
@tanmoysrt
Copy link
Member Author

tanmoysrt commented Feb 10, 2023 •
edited
Loading

@tachyons @vedant-jain03 @satu0king 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

@tachyons tachyons tachyons approved these changes

@satu0king satu0king Awaiting requested review from satu0king

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tanmoysrt @tachyons