-
Notifications
You must be signed in to change notification settings - Fork 54
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
Added Theme switch button #192
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please follow the instructions given in pr template more carefully
Provide a demo link
Also follow the contribution guidelines
@pranav1698 Could you suggest any way to provide a demo link.. |
@pranav1698 I did as suggested but it’s not working. |
You can find the instructions in the readme file for web hosting |
I had done it and my link is |
Refer readme file it is a mozilla addon |
@pranav1698 Please check it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NishithP2004 you need to share the signed version of the add-on not the zip File
@pranav1698 I’m not able to sign it as it says a similar add on is available and hence error. |
@Orbiter @mariobehling Please review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NishithP2004 you can check these links: https://support.mozilla.org/en-US/kb/unable-install-add-ons-extensions-or-themes
https://support.mozilla.org/en-US/kb/cannot-remove-add-on-extension-or-theme
https://support.mozilla.org/en-US/questions/1101877
We can help you to sign add-ons but we need to test it to aproove the pr
Fixes #118
Changes proposed in this pull request:
Demo Link:
[Add here the link where your changes can be seen]
Screenshots for the change: