Skip to content

Commit

Permalink
address some PR comments
Browse files Browse the repository at this point in the history
  • Loading branch information
tomkralidis committed Apr 2, 2024
1 parent a0dce49 commit 35b14dd
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion pygeoapi/openapi.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@
THISDIR = os.path.dirname(os.path.realpath(__file__))


def get_ogc_schemas_location(server_config) -> dict:
def get_ogc_schemas_location(server_config: dict) -> str:
"""
Determine OGC schemas location
Expand Down
2 changes: 0 additions & 2 deletions pygeoapi/process/manager/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -320,8 +320,6 @@ def _execute_handler_sync(self, p: BaseProcessor, job_id: str,
self.update_job(job_id, job_metadata)
self._send_failed_notification(subscriber)

self._send_failed_notification(subscriber)

return jfmt, outputs, current_status

def execute_process(
Expand Down
5 changes: 4 additions & 1 deletion pygeoapi/util.py
Original file line number Diff line number Diff line change
Expand Up @@ -609,9 +609,12 @@ class JobStatus(Enum):

@dataclass(frozen=True)
class Subscriber:
"""Store subscriber urls as defined in:
"""
Store subscriber URLs as defined in:
https://schemas.opengis.net/ogcapi/processes/part1/1.0/openapi/schemas/subscriber.yaml # noqa
"""

success_uri: str
in_progress_uri: Optional[str]
failed_uri: Optional[str]
Expand Down

0 comments on commit 35b14dd

Please sign in to comment.