Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Changed service name for document in docker compose#17

Merged
hari-kuriakose merged 1 commit into
mainZipstack/unstract:mainfrom
fix/document-service-name-dockerZipstack/unstract:fix/document-service-name-dockerCopy head branch name to clipboard
Feb 29, 2024
Merged

Changed service name for document in docker compose#17
hari-kuriakose merged 1 commit into
mainZipstack/unstract:mainfrom
fix/document-service-name-dockerZipstack/unstract:fix/document-service-name-dockerCopy head branch name to clipboard

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.