Skip to content

Commit

Permalink
Merge pull request apache#27 from riskive/check-api-respond
Browse files Browse the repository at this point in the history
Check api respond
  • Loading branch information
rguerraZero authored Sep 6, 2023
2 parents 1db3771 + ad0975a commit 4001071
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions zf_integration/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@

from typing import Any, Dict

from flask import request, Response
from flask import request, Response, jsonify, make_response
from flask_appbuilder import expose
from marshmallow import ValidationError
from superset.utils.jwt import JWTParser
Expand Down Expand Up @@ -93,8 +93,8 @@ def get_info(self) -> Response:
'guest_token': guest_token,
'default_dashboards': default_dashboards['dashboards'],
'custom_dashboards': custom_dashboards['dashboards'],
}
return self.response(200, data=respond)
}
return make_response(jsonify(respond), 200)
except ValidationError as error:
return self.response_400(message=error.messages)

Expand Down Expand Up @@ -200,7 +200,7 @@ def get_dashboards_details_by_roles(self, roles) -> Dict[str, Any]:
Dashboard.published.is_(True),
Role.id.in_(roles),
),
).all()
).distinct().all()
data = {
'uuids': [str(d.uuid) for d in dashboards],
'dashboards': [{'id': d.id, 'name': d.dashboard_title, 'uuid': d.uuid} for d in dashboards]
Expand Down

0 comments on commit 4001071

Please sign in to comment.