Skip to content

Commit

Permalink
Merge pull request #139 from CartoDB/bug/ch99519/dataview-formuladata…
Browse files Browse the repository at this point in the history
…view-dataviewremote-bugs

bug fixed
  • Loading branch information
padawannn authored Sep 2, 2020
2 parents b866967 + 54530a6 commit ac84d20
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/viz/dataview/formula/FormulaDataViewImpl.ts
Original file line number Diff line number Diff line change
Expand Up @@ -59,14 +59,15 @@ export class FormulaDataViewImpl extends DataViewImpl<FormulaDataViewData> {
try {
const filterApplicator = dataviewRemote.getFilterApplicator();
const bbox = filterApplicator.getBbox();

dataviewRemote.updateDataViewSource(options);

const dataviewsApiResponse = await dataviewRemote.dataviewsApi.formula({
column: this.column,
operation: this.operation,
bbox
});

dataviewRemote.updateDataViewSource(options);

if (
dataviewsApiResponse.errors_with_context &&
dataviewsApiResponse.errors_with_context.length > 0
Expand Down

1 comment on commit ac84d20

@vercel
Copy link

@vercel vercel bot commented on ac84d20 Sep 2, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.