Skip to content

Scandoc.ai Implementation#15

Open
marinm03 wants to merge 5 commits intodevelopmentfrom
mm/scandoc-implementation
Open

Scandoc.ai Implementation#15
marinm03 wants to merge 5 commits intodevelopmentfrom
mm/scandoc-implementation

Conversation

@marinm03
Copy link
Copy Markdown
Contributor

@marinm03 marinm03 commented Feb 9, 2026

No description provided.

@marinm03 marinm03 requested a review from kskunca314 February 9, 2026 12:07
@marinm03 marinm03 self-assigned this Feb 9, 2026
@kskunca314
Copy link
Copy Markdown
Contributor

Izmijeniti imena exposanih metoda i classa da odgovaraju nativnim metodama.
Izdvojiti modele u zasebne klase.
SCANDOC_SCAN_BASE_URL mora biti konfigurabilan.

Comment thread example/src/App.tsx Outdated
Comment thread example/src/App.tsx Outdated
Comment thread example/src/App.tsx Outdated
Comment thread example/src/App.tsx Outdated
Comment thread src/hooks/useAuth.ts
@@ -0,0 +1,81 @@
import { useCallback, useState, useRef } from 'react';
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.

ne znam pravila react nativea i ako mora biti use ispred ali ime klase mi apsolutnu nista ne govori. promijeni ime u useScanDocAuthentication.

Comment thread src/hooks/useAuth.ts Outdated
Comment thread src/hooks/useScanDoc.ts Outdated
Comment thread src/hooks/useScanDoc.ts Outdated
Comment thread src/types/scandocTypes.ts
@@ -0,0 +1,75 @@
export type ScandocAuthResponse = Record<string, unknown>;
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.

index je neki stnadard za modele ili? modeli trebaju biti odvojeni, svaki zasebno

Comment thread src/api.ts Outdated
@@ -0,0 +1,114 @@
import {
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.

Preimenovati u ScanDocApiHtmlImpl, probaj pratiti nativno da imamo svuda 1 na 1

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.

2 participants