diff --git a/src/api/admin.js b/src/api/admin.js index 79eac9bd6..67fb5038b 100644 --- a/src/api/admin.js +++ b/src/api/admin.js @@ -312,9 +312,6 @@ export const deleteAnnouncement = ({ id, credentials }) => }) export const setReportState = ({ id, state, credentials }) => { - // TODO: Can't use promisedRequest because on OK this does not return json - // See https://git.pleroma.social/pleroma/pleroma-fe/-/merge_requests/1322 - return promisedRequest({ url: REPORTS, credentials, @@ -328,19 +325,6 @@ export const setReportState = ({ id, state, credentials }) => { ], }, }) - .then((data) => { - if (data.status >= 500) { - throw Error(data.statusText) - } else if (data.status >= 400) { - return data.json() - } - return data - }) - .then((data) => { - if (data.errors) { - throw Error(data.errors[0].message) - } - }) } export const getInstanceDBConfig = ({ credentials }) => diff --git a/src/api/user.js b/src/api/user.js index ce41c0c21..b9035da77 100644 --- a/src/api/user.js +++ b/src/api/user.js @@ -842,7 +842,6 @@ export const getList = ({ listId, credentials }) => }) export const updateList = ({ listId, title, credentials }) => - console.log('PUT', MASTODON_LIST_URL(listId)) || promisedRequest({ url: MASTODON_LIST_URL(listId), diff --git a/src/components/post_status_form/post_status_form.js b/src/components/post_status_form/post_status_form.js index 4d9c34546..6cb99a3b6 100644 --- a/src/components/post_status_form/post_status_form.js +++ b/src/components/post_status_form/post_status_form.js @@ -634,6 +634,7 @@ const PostStatusForm = { // Don't apply preview if not loading, because it means // user has closed the preview manually. if (!this.previewLoading) return + this.preview = data }) .catch((error) => { this.preview = { error } diff --git a/src/components/user_reporting_modal/user_reporting_modal.js b/src/components/user_reporting_modal/user_reporting_modal.js index 9659022d4..017a17efa 100644 --- a/src/components/user_reporting_modal/user_reporting_modal.js +++ b/src/components/user_reporting_modal/user_reporting_modal.js @@ -31,7 +31,6 @@ const UserReportingModal = { return !!this.$store.state.users.currentUser }, isOpen() { - console.log(this.reportModal) return this.isLoggedIn && this.reportModal.activated }, userId() { diff --git a/src/services/errors/errors.js b/src/services/errors/errors.js index a28f31775..ccbae9b3e 100644 --- a/src/services/errors/errors.js +++ b/src/services/errors/errors.js @@ -13,9 +13,9 @@ function humanizeErrors(errors) { export function StatusCodeError(statusCode, body, options, response) { this.name = 'StatusCodeError' this.statusCode = statusCode - this.statusText = body.error || body + this.statusText = body.error || body.errors || body this.details = JSON && JSON.stringify ? JSON.stringify(body) : body - this.errorData = body.error + this.errorData = body.error || body.errors this.message = this.statusCode + ' - ' + this.statusText this.error = body // legacy attribute this.options = options diff --git a/src/stores/oauth_tokens.js b/src/stores/oauth_tokens.js index 10aa371f9..4d18ffae7 100644 --- a/src/stores/oauth_tokens.js +++ b/src/stores/oauth_tokens.js @@ -4,6 +4,11 @@ import { useOAuthStore } from 'src/stores/oauth.js' import { fetchOAuthTokens, revokeOAuthToken } from 'src/api/user.js' +/* Just to clear the confusion: + * OAuth Store is responsible for user authentication + * OAuth Tokens Store is responsible for *managing* all of the user's tokens, + * i.e. for current and other clients + */ export const useOAuthTokensStore = defineStore('oauthTokens', { state: () => ({ tokens: [], diff --git a/test/fixtures/worker.js b/test/fixtures/worker.js index e0db13ffc..e6ed89dc9 100644 --- a/test/fixtures/worker.js +++ b/test/fixtures/worker.js @@ -2,7 +2,4 @@ import { setupWorker } from 'msw/browser' export const worker = setupWorker() -console.log('=============== TEST ===============') -console.log(window.__test__) -console.log('=============== TEST ===============') window.__test__ = window.__test__ || 'TEST' diff --git a/test/unit/specs/stores/lists.spec.js b/test/unit/specs/stores/lists.spec.js index 3720aa776..bb1ef12b4 100644 --- a/test/unit/specs/stores/lists.spec.js +++ b/test/unit/specs/stores/lists.spec.js @@ -36,7 +36,6 @@ describe('The lists store', () => { HttpResponse.json({ ok: true }), ), ) - console.log('1 =========', worker.listHandlers()) await store.setList({ listId: list.id, title: list.title }) expect(store.allListsObject[list.id]).to.eql({ @@ -46,8 +45,6 @@ describe('The lists store', () => { expect(store.allLists).to.have.length(1) expect(store.allLists[0]).to.eql(list) - console.log('2 =========', worker.listHandlers()) - await store.setList({ listId: modList.id, title: modList.title }) expect(store.allListsObject[modList.id]).to.eql({ title: modList.title, @@ -55,8 +52,6 @@ describe('The lists store', () => { }) expect(store.allLists).to.have.length(1) expect(store.allLists[0]).to.eql(modList) - - console.log('3 =========', worker.listHandlers()) }) it('adds a new list with an array of IDs, updating the IDs for existing lists', async ({ diff --git a/test/unit/specs/stores/oauth.spec.js b/test/unit/specs/stores/oauth.spec.js index 6e61fea79..a06cbb2fd 100644 --- a/test/unit/specs/stores/oauth.spec.js +++ b/test/unit/specs/stores/oauth.spec.js @@ -155,7 +155,6 @@ describe('oauth store', () => { describe('ensureAppToken', () => { it('should work if the state is empty', async ({ worker }) => { worker.use(...authApis()) - console.log('=========', worker.listHandlers()) const store = useOAuthStore() const token = await store.ensureAppToken() expect(token).to.eql('test-app-token')