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

fix: 🚧 show something when dr without inst_id #394

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jalezi
Copy link
Collaborator

@jalezi jalezi commented Oct 24, 2023

If doctor has no inst_id we should show something instead of 404 page.

Something like this, but without typos 😄.
Screenshot 2023-10-24 at 08 00 22

@jalezi jalezi marked this pull request as draft October 24, 2023 05:50
@SterArcher
Copy link
Contributor

i would change this to:
Ta zdravnik se ne nahaja v našem zajemu podatku. Prosimo preverite ali se zdravnik nahaja v bazi ZZZS, saj je ta baza osnova za naš zajem podatkov. V primeru, da je tam zdravnik prisoten počakajte nekaj dni, da se spremembe prikažejo pri nas. V nasprotnem primeru prosimo kontaktirajte [email protected]. Hvala!

       Ta zdravnik se ne nahaja v našem zajemu podatku. 
            Prosimo preverite ali se zdravnik nahaja v bazi ZZZS, saj je ta baza osnova za naš zajem podatkov. 
            V primeru, da je tam zdravnik prisoten počakajte nekaj dni, da se spremembe prikažejo pri nas. 
            V nasprotnem primeru prosimo kontaktirajte [email protected]. 
            Hvala!
@SterArcher SterArcher requested a review from stefanb March 10, 2024 14:40
@SterArcher SterArcher added the enhancement New feature or request label Mar 10, 2024
Applied some styling for no instId error.
Add mailto link.
@jalezi
Copy link
Collaborator Author

jalezi commented Mar 12, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy-preview enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants