This repository has been archived by the owner on Nov 21, 2022. It is now read-only.
0.1alpha initial commit #79
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…n to change as background -added babel for testing purposes -separated restoreOptions function -added basic skeleton for function tests -separated tests and assets in proper folders -updated onError function -added selenium setup -added selenium functionality test - (Work in progress) -updates popup functions -updated text_white styles -modified .eslintrc.yml -updated package.json -updated StyleTweet function -minor code changes for better readability
Since selenium tests seem to be the tail end of the work, we can tackle them in a separate PR. Signed-off-by: Muaaz Saleem <[email protected]>
…ific-update_to_tests_and_code Taking out selenium tests so we can merge the rest of the PR
…istic` buttons -disabled all other functionality except crossed-trough -added injectReportMisogynisticButtons function -found and inspected issue with memory leak -fixed memory leak issue caused by repeated sending of already sent server requests -added css for reportMisogynisticButtons and reportMisogynisticButtonsHolder -added isn't and is misogynistic buttons creation in injectReportMisogynisticButtons function -added event listener for clicks on document from reportMisogynisticButtons -added reportMisogyny function for sending request for reporting tweet -added permission in manifest.json for reporting tweet api -fixed bug where text sent to prediction api has html code
-fixed memory leak issue caused by repeated sending of already sent server requests -fixed bug where text sent to prediction api has html code
-add tweetId to sent object for api endpoint -fixed hide/unhide locally
…ned_text' into #78_bug_fix_memory_leak_&_uncleaned_text # Conflicts: # extension/content/opt-out-ext.js
…ix_memory_leak_&_uncleaned_text opt-out-tools#78 bug fix memory leak & uncleaned text
# Conflicts: # extension/content/opt-out-ext.js
# Conflicts: # extension/content/opt-out-ext.js
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
make text invisibleandcompletely remove options-merged commits from latest stable master version with bug fixes to this branch
resolves issue #66