diff --git a/docs/postBuilder_MastodonPostBuilder.js.html b/docs/postBuilder_MastodonPostBuilder.js.html
index 602e571..9b29401 100644
--- a/docs/postBuilder_MastodonPostBuilder.js.html
+++ b/docs/postBuilder_MastodonPostBuilder.js.html
@@ -29,22 +29,6 @@ Source: postBuilder/MastodonPostBuilder.js
import { PostBuilder } from "./PostBuilder.js";
export class MastodonPostBuilder extends PostBuilder {
- // TODO: review this function, remove if not needed
- #interleaveArrays(responses) {
- const maxLength = Math.max(...responses.map(response => response.length));
- const result = [];
-
- for (let i = 0; i < maxLength; i++) {
- for (let j = 0; j < responses.length; j++) {
- if (responses[j].length > i) {
- result.push(responses[j][i]);
- }
- }
- }
-
- return result;
- }
-
/**
*
* @param {Object} rawPost - The raw json data from the API
@@ -78,7 +62,7 @@ Classes
-
diff --git a/docs/postBuilder_PostBuilder.js.html b/docs/postBuilder_PostBuilder.js.html
index ca1713d..12aa3e6 100644
--- a/docs/postBuilder_PostBuilder.js.html
+++ b/docs/postBuilder_PostBuilder.js.html
@@ -57,7 +57,7 @@
Classes
-
diff --git a/specs/project specifications.md b/specs/project specifications.md
index d1701b9..133cab4 100644
--- a/specs/project specifications.md
+++ b/specs/project specifications.md
@@ -60,12 +60,4 @@ The project's feature goals are as follows:
- Advanced recommendation algorithm
- More platforms (PeerTube, Pixelfed, etc.)
- Customizable UI themes
-- A way to share customized instance lists among devices
-
-
-## Project architecture
-TODO: insert diagram
-
-## Wireframes and ideation
-TODO: insert screenshots from Miro
-
+- A way to share customized instance lists among devices
\ No newline at end of file
diff --git a/src/scripts/entity/InstanceEntry.js b/src/scripts/entity/InstanceEntry.js
index 726590e..cfcea31 100644
--- a/src/scripts/entity/InstanceEntry.js
+++ b/src/scripts/entity/InstanceEntry.js
@@ -56,10 +56,6 @@ export class InstanceEntry extends HTMLElement {
script.type = "module";
script.textContent = `
import { handleRemoveInstance } from './src/scripts/instanceList.js';
- // document.getElementById('instance-removal-button').addEventListener('click', () => {
- // handleRemoveInstance(${this.network}, ${this.url});
- // console.log('removal button clicked');
- // });
window.handleRemoveInstance = handleRemoveInstance;
`;
this.shadowRoot.appendChild(script);
diff --git a/src/scripts/instanceList.js b/src/scripts/instanceList.js
index 9a46b15..b38b9e4 100644
--- a/src/scripts/instanceList.js
+++ b/src/scripts/instanceList.js
@@ -43,15 +43,7 @@ export function saveLists(instanceLists, storage = localStorage) {
* @returns {Promise} Promise which resolves to True if the instance addition was successful, False otherwise
*/
export async function addInstance(network, url, storage = localStorage) {
- // TODO: move validation out of this function? (may separate concerns better)
if (!(ALLOWED_NETWORKS.has(network)) || !validUrl(url)) return false;
- // Commented out because this creates CORS errors if not done exactly on API
- // try {
- // let response = await fetch(url);
- // if (response.status !== 200) return false;
- // } catch (_) {
- // return false;
- // }
let instanceList = fetchInstanceLists(storage);
// network is already guaranteed to be in ALLOWED_NETWORKS and thus on the default list
diff --git a/src/scripts/pageBuilder.js b/src/scripts/pageBuilder.js
index 4fa181b..debd355 100644
--- a/src/scripts/pageBuilder.js
+++ b/src/scripts/pageBuilder.js
@@ -19,7 +19,6 @@ export async function buildPage(HANDLERS) {
nextPage.disabled = true;
prevPage.disabled = true;
- // TODO: sane pre-fetching solution that isn't just one instance
let postsByNetwork = {};
let maxLengthArray = 0;
const instLists = fetchInstanceLists();
diff --git a/src/scripts/postBuilder/MastodonPostBuilder.js b/src/scripts/postBuilder/MastodonPostBuilder.js
index 281ba08..c5a9855 100644
--- a/src/scripts/postBuilder/MastodonPostBuilder.js
+++ b/src/scripts/postBuilder/MastodonPostBuilder.js
@@ -1,22 +1,6 @@
import { PostBuilder } from "./PostBuilder.js";
export class MastodonPostBuilder extends PostBuilder {
- // TODO: review this function, remove if not needed
- #interleaveArrays(responses) {
- const maxLength = Math.max(...responses.map(response => response.length));
- const result = [];
-
- for (let i = 0; i < maxLength; i++) {
- for (let j = 0; j < responses.length; j++) {
- if (responses[j].length > i) {
- result.push(responses[j][i]);
- }
- }
- }
-
- return result;
- }
-
/**
*
* @param {Object} rawPost - The raw json data from the API
diff --git a/src/scripts/settings-page.js b/src/scripts/settings-page.js
index c399a7e..6a319ea 100644
--- a/src/scripts/settings-page.js
+++ b/src/scripts/settings-page.js
@@ -31,7 +31,6 @@ addInstBtns.forEach((element) => {
// add https:// if the user forgot
if (!(url.includes("//"))) {
url = "https://".concat(url);
- console.log(url);
}
let success = await addInstance(network, url);
if (success) {
@@ -43,14 +42,12 @@ addInstBtns.forEach((element) => {
input.value = ""; // clear input box after adding to UI
}
else {
- // TODO: don't use an alert
alert("Adding instance failed, please try again");
}
});
});
// Add event listeners for "reset to default"
-// TODO: refactor CSS classes to "button" or "btn". choose one
let resetDefaultBtns = Array.from(document.getElementsByClassName("instances-reset-button"));
resetDefaultBtns.forEach((element) => {
element.addEventListener("click", (event) => {