Skip to content

Commit

Permalink
Fix typo + add todo
Browse files Browse the repository at this point in the history
  • Loading branch information
m-mohr committed Jul 29, 2024
1 parent b8327dd commit 7fcfd38
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
3 changes: 2 additions & 1 deletion src/api/collections.js
Original file line number Diff line number Diff line change
Expand Up @@ -151,6 +151,7 @@ export default class Data {
const limit = parseInt(req.query.limit, 10) || 10;
const offset = parseInt(req.query.offset, 10) || 0;

// todo: migrate to ee.data.listImages?

Check warning on line 154 in src/api/collections.js

View workflow job for this annotation

GitHub Actions / deploy (17)

Unexpected 'todo' comment: 'todo: migrate to ee.data.listImages?'

Check warning on line 154 in src/api/collections.js

View workflow job for this annotation

GitHub Actions / deploy (lts/*)

Unexpected 'todo' comment: 'todo: migrate to ee.data.listImages?'

Check warning on line 154 in src/api/collections.js

View workflow job for this annotation

GitHub Actions / deploy (latest)

Unexpected 'todo' comment: 'todo: migrate to ee.data.listImages?'
// Load the collection
let ic = this.ee.ImageCollection(id);

Expand Down Expand Up @@ -219,7 +220,7 @@ export default class Data {
}

// Limit
const icList = ic.toList(limit + 1, offset)
const icList = ic.toList(limit + 1, offset);

// Retrieve the items
let items;
Expand Down
6 changes: 3 additions & 3 deletions src/api/jobs.js
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ export default class JobsAPI {
token: req.params.token
};

await this.getJobRultsByQuery(query, true, req, res);
await this.getJobResultsByQuery(query, true, req, res);
}

async getJobResults(req, res) {
Expand All @@ -153,7 +153,7 @@ export default class JobsAPI {
_id: req.params.job_id,
user_id: req.user._id
};
await this.getJobRultsByQuery(query, false, req, res);
await this.getJobResultsByQuery(query, false, req, res);
}

async deleteJobResults(req, res) {
Expand All @@ -175,7 +175,7 @@ export default class JobsAPI {
res.send(204);
}

async getJobRultsByQuery(query, publish, req, res) {
async getJobResultsByQuery(query, publish, req, res) {
const job = await this.storage.findJob(query);
const partial = typeof req.query.partial !== 'undefined';
if (job.status === 'error') {
Expand Down

0 comments on commit 7fcfd38

Please sign in to comment.