Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Novi potpis zdravstvene knjižice #34

Merged
merged 2 commits into from
Sep 9, 2024
Merged

Conversation

RockyMM
Copy link

@RockyMM RockyMM commented Sep 7, 2024

Hteo sam da odštampam podatke sa ćerkine zdravstvene knjižice i primetio sam da nije hteo da očita karticu. Ispostavilo se da ima novi potpis.

Usput sam izmenio neke UI stvari, nadam se na bolje; pošto mi je trebalo da imam broj zdravstvene isprave na ekranu i u PDF-u.

@ubavic
Copy link
Owner

ubavic commented Sep 8, 2024

Zdravo.

Hvala na PR-u!

Zanima me samo da li se na štampi te z. knjižice pojavljuje broj zdravstvene isprave? Pošto ako se ne pojavljuje, onda ću vratiti štampu kako je bila (cilj je da PDF ostane identičan PDF-u dobijenim za zvaničnom aplikacijom). Što se tiče interfejsa, tu nema problema, broj može da ostane.

Edit: ako se ne pojavljuje, molim te ukloni promene za PDF, kao i promene i FyneApp.toml datoteci pa ću merdžovati (idealno sa git ammend-om). Hvala još jednom!

@RockyMM
Copy link
Author

RockyMM commented Sep 9, 2024

Pozdrav @ubavic ,

Izmenio sam kako si opisao. Inače, ovo je moj prvi susret s go-om, tako da sam malo lutao kako da se snađem, ali sam se snašao :)

P.S. izmenio sam “generateVersion.go“ jer sam hteo da imam verziju aplikacije i u lokalu. To mogu da napravim PR drugom prilikom :)

P.P.S. Razumem da PDF treba da ostane identičan zvančinom, ali je meni baš bilo potrebno da vidim broj kartice i napismeno. Naravno, to sam odštampao za sopstvene potrebe. Ali ne razumem kako im je takav papir koristan a da nema broj kartice. A i drugo, taj vebservis za osvežavanje datuma overe bi mogao da prosledi i podatke o osiguraocu, pošto kod mene prikazuje samo prvo pravno lice preko kojeg je išlo osiguranje, a ne preko kojeg trenutno ide osiguranje.

Ali to je država... ima neku svoju logiku :D

@ubavic ubavic merged commit 2e2bf88 into ubavic:main Sep 9, 2024
@ubavic
Copy link
Owner

ubavic commented Sep 9, 2024

Merdžovano. Hvala.

Izmenio sam kako si opisao. Inače, ovo je moj prvi susret s go-om, tako da sam malo lutao kako da se snađem, ali sam se snašao :)

Odlično je ovo odrađeno 👍 Ako imaš neke komentare kako mogu da olakšam snalaženje u projektu, slobodno piši.

A i drugo, taj vebservis za osvežavanje datuma overe bi mogao da prosledi i podatke o osiguraocu, pošto kod mene prikazuje samo prvo pravno lice preko kojeg je išlo osiguranje, a ne preko kojeg trenutno ide osiguranje.

Ovo je koristan komentar, i sugeriše da postoji bug u Baš Čeliku. Zvanična aplikacija koristi jedan servis namenjen samo za aplikaciju. Baš Čelik koristi javni veb servis. Moguće je da postoji razlika između načina funkcionisanja ova dva servisa (tj razlika između podataka koji se vraćaju).

PS. generateVersion.go planiram već dugo da uklonim, jer je shell skripta dovoljna. Šta si planirao da ubaciš u taj PR?

@RockyMM
Copy link
Author

RockyMM commented Sep 9, 2024

Za generateVersion.go sam mislio nešto ovako:

package main

import (
	"github.com/Masterminds/semver/v3"
	"os"
	"os/exec"
	"strings"
)

func main() {
	cmd := exec.Command("git", "describe", "--tags")
	stdout, err := cmd.Output()
	if err != nil {
		return
	}

	tag := strings.TrimSpace(string(stdout))
	versionSemver, err := semver.NewVersion(tag)
	if err != nil {
		return
	}

	var version []byte = nil
	if versionSemver.Prerelease() != "" {
		workingSemver, _ := versionSemver.SetPrerelease("")
		workingSemver = workingSemver.IncPatch()
		versionSemverWithAlpha, err := workingSemver.SetPrerelease("alpha")
		if err != nil {
			return
		}
		versionSemver = &versionSemverWithAlpha
	}
	version = []byte(versionSemver.String())

	err = os.WriteFile("assets/version", version, 0600)
	if err != nil {
		return
	}
}

U suštini, hteo sam da piše -alpha I da se verzija uveća sama za jedan :D Više je bilo za moje vežbanje go-a nego nešto produktivno.

A glede RFZO servisa, ne postoji bug u Baš Čelku, već se servisi razlikuju... 🤷‍♂️

@RockyMM RockyMM deleted the feature/nova-zdr branch September 12, 2024 14:57
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