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.

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

Implement bug reporting logic #3000

Merged
merged 20 commits into from Jan 25, 2017
Merged

Implement bug reporting logic #3000

Show file tree
Hide file tree
Changes from 15 commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
e988476
Persist console logs to an IndexedDB instance
kegsay Jan 18, 2017
5828ab1
Generate unique IDs for each JS runtime to accomodate multiple tabs
kegsay Jan 18, 2017
811086a
Comment how we should consume logs
kegsay Jan 18, 2017
89d514a
Query IndexedDB for all results with index=0
kegsay Jan 19, 2017
608c1b0
Finish implementing consume()
kegsay Jan 19, 2017
81d437a
POST reports to localhost for now. Also send live console logs
kegsay Jan 19, 2017
bf887e8
Swap to async/await rather than promise chains
kegsay Jan 19, 2017
3996d23
Inject bug report endpoint URL from config.json
kegsay Jan 19, 2017
beba4d2
Add version and user agent to bug report
kegsay Jan 19, 2017
53a1615
Remove logs when submitting bug reports
kegsay Jan 19, 2017
ea86080
Add a monkeyPatch function rather than monkey-patching in the constru…
kegsay Jan 20, 2017
ba1e166
Line length 80 like we're still in the 80s
kegsay Jan 20, 2017
41c6294
Remove clearAll from consume(): we want duplicate logs on multiple re…
kegsay Jan 20, 2017
378126e
Add another store to keep last modified times
kegsay Jan 20, 2017
ea063ab
Address race conditions when flushing logs
kegsay Jan 20, 2017
2ecf65f
Keep the logs if no store exists
kegsay Jan 23, 2017
f3df868
Fix tightlooping when flush()ing without any logs
kegsay Jan 23, 2017
6f3b70d
Use Q promises and isPending to make logic simpler
kegsay Jan 24, 2017
e225d3e
Preserve ordering of flush()es by not letting subsequent flush()es race
kegsay Jan 24, 2017
eaff0ab
Add link to Chrome issue status for fixing line numbers
kegsay Jan 25, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions config.sample.json
View file
Open in desktop
Expand Up @@ -4,6 +4,7 @@
"brand": "Riot",
"integrations_ui_url": "https://scalar.vector.im/",
"integrations_rest_url": "https://scalar.vector.im/api",
"bug_report_endpoint_url": "https://vector.im/bugs",
"enableLabs": true,
"roomDirectory": {
"servers": [
Expand Down
9 changes: 9 additions & 0 deletions src/vector/index.js
View file
Open in desktop
Expand Up @@ -36,6 +36,14 @@ require('gfm.css/gfm.css');
require('highlight.js/styles/github.css');
require('draft-js/dist/Draft.css');

const rageshake = require("./rageshake");
rageshake.init().then(() => {
console.log("Initialised rageshake");
rageshake.cleanup();
}, (err) => {
console.error("Failed to initialise rageshake: " + err);
});


// add React and ReactPerf to the global namespace, to make them easier to
// access via the console
Expand Down Expand Up @@ -233,6 +241,7 @@ async function loadApp() {
let configError;
try {
configJson = await getConfig();
rageshake.setBugReportEndpoint(configJson.bug_report_endpoint_url);
} catch (e) {
configError = e;
}
Expand Down