Skip to content

Changed service name for document in docker compose#17

Merged
hari-kuriakose merged 1 commit into
mainfrom
fix/document-service-name-docker
Feb 29, 2024
Merged

Changed service name for document in docker compose#17
hari-kuriakose merged 1 commit into
mainfrom
fix/document-service-name-docker

Conversation

@ritwik-g

@ritwik-g ritwik-g commented Feb 29, 2024

Copy link
Copy Markdown
Contributor

What

  • Changed service name for document in docker compose

Why

  • So that the name matches with the one used in docker-compse.build.yaml and also with the existing convention we follow

How

...

Relevant Docs

Related Issues or PRs

Dependencies Versions / Env Variables

Notes on Testing

...

Screenshots

...

Checklist

I have read and understood the Contribution Guidelines.

Signed-off-by: Ritwik G <100672805+ritwik-g@users.noreply.github.com>

@hari-kuriakose hari-kuriakose left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ritwik-g LGTM

@hari-kuriakose hari-kuriakose merged commit 8b263ed into main Feb 29, 2024
@hari-kuriakose hari-kuriakose deleted the fix/document-service-name-docker branch February 29, 2024 06:38
ritwik-g added a commit that referenced this pull request Feb 29, 2024
…19)

* Basic workflow and adapter exception messages

* Corrected alignment of the OnBoard page to accommodate various screen sizes

* Adapter error messages updated

* removed translator, PII redactor and workflow settings

* ocr tool with adapter support

* Changed service name for document in docker compose (#17)

Signed-off-by: Ritwik G <100672805+ritwik-g@users.noreply.github.com>

* PR comments addressed
sdk version update in requirements
ocr_adapter.process() used to write output
readme updated
tools json typo fix

---------

Signed-off-by: ali <117142933+muhammad-ali-e@users.noreply.github.com>
Signed-off-by: Ritwik G <100672805+ritwik-g@users.noreply.github.com>
Co-authored-by: Athul <athul@zipstack.com>
Co-authored-by: siddhiq <siddhiq@zipstack.com>
Co-authored-by: Siddhiq <113336661+mohamed-siddhiq@users.noreply.github.com>
Co-authored-by: Athul <89829560+athul-rs@users.noreply.github.com>
Co-authored-by: ali-zipstack <muhammad.ali@zipstack.com>
Co-authored-by: vishnuszipstack <vishnu@zipstack.com>
Co-authored-by: ali <117142933+muhammad-ali-e@users.noreply.github.com>
Co-authored-by: vishnuszipstack <117254672+vishnuszipstack@users.noreply.github.com>
Co-authored-by: Neha <115609453+nehabagdia@users.noreply.github.com>
ritwik-g added a commit that referenced this pull request Feb 29, 2024
* Basic workflow and adapter exception messages

* Corrected alignment of the OnBoard page to accommodate various screen sizes

* Adapter error messages updated

* removed translator, PII redactor and workflow settings

* ocr tool with adapter support

* Changed service name for document in docker compose (#17)

Signed-off-by: Ritwik G <100672805+ritwik-g@users.noreply.github.com>

* PR comments addressed
sdk version update in requirements
ocr_adapter.process() used to write output
readme updated
tools json typo fix

* Added docker fixes

---------

Signed-off-by: ali <117142933+muhammad-ali-e@users.noreply.github.com>
Signed-off-by: Ritwik G <100672805+ritwik-g@users.noreply.github.com>
Co-authored-by: Athul <athul@zipstack.com>
Co-authored-by: siddhiq <siddhiq@zipstack.com>
Co-authored-by: Siddhiq <113336661+mohamed-siddhiq@users.noreply.github.com>
Co-authored-by: Athul <89829560+athul-rs@users.noreply.github.com>
Co-authored-by: ali-zipstack <muhammad.ali@zipstack.com>
Co-authored-by: vishnuszipstack <vishnu@zipstack.com>
Co-authored-by: ali <117142933+muhammad-ali-e@users.noreply.github.com>
Co-authored-by: vishnuszipstack <117254672+vishnuszipstack@users.noreply.github.com>
Co-authored-by: Neha <115609453+nehabagdia@users.noreply.github.com>
pk-zipstack pushed a commit that referenced this pull request Aug 20, 2025
Signed-off-by: Ritwik G <100672805+ritwik-g@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants