Skip to content

Render error tx844 #898

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.

Sign up for GitHub

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

Open
sohammk08 wants to merge 12 commits into
base: staging
Choose a base branch
from
Open

Render error tx844 #898

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
9db41c9
refactor: translations.json
sohammk08 Dec 19, 2023
908494f
Merge branch 'ripple:staging' into render-error-tx844
sohammk08 Dec 19, 2023
32ef2c6
Merge branch 'ripple:staging' into render-error-tx844
sohammk08 Jan 20, 2024
f1f03d1
Merge branch 'staging' into render-error-tx844
sohammk08 Feb 1, 2024
59d96b7
Merge branch 'ripple:staging' into render-error-tx844
sohammk08 Feb 3, 2024
8dbf4b8
Merge branch 'ripple:staging' into render-error-tx844
sohammk08 Mar 9, 2024
e619909
This commit is made to try and solve a issue in vercel deployment
sohammk08 Apr 3, 2024
7839eb0
pubKey displays properly
sohammk08 Apr 3, 2024
2c74698
Merge branch 'staging' into render-error-tx844
sohammk08 Apr 6, 2024
5470806
Resolved Issue #844
sohammk08 Apr 6, 2024
df54121
Merge branch 'render-error-tx844' of https://github.com/sohammk08/exp…
sohammk08 Apr 6, 2024
d05d2e1
refactor NoMatch index.tsx
sohammk08 Apr 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion public/locales/en-US/translations.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@
"invalid_transaction_hash": "The transaction hash is invalid",
"ledger_not_found": "Ledger not Found",
"check_ledger_id": "Please check your ledger id",
"server_ledgers_hint": "This node ({{connection.server.publicKey, truncate(length: 10)}}) only contains ledgers {{connection.ledger.validated}}",

@mvadari mvadari Jan 4, 2024

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think we still want the pubkey to be printed if it's available, with some sort of "This Clio node only..." alternate message

@sohammk08 sohammk08 Jan 5, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

So, you're saying that the clio-node-part still needs to be present in the code. What if I change the code such that whenever the pubKey is not empty, it gets displayed. But when it is empty, print it as "This clio node only..."

Would that be viable?

@mvadari mvadari Jan 31, 2024

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yes, something like that sounds good.

"server_ledgers_hint": "This node only contains ledgers {{connection.ledger.validated}}",
"use_search": "Please use our search",
"ledger_has_no_trans": "This ledger does not have any transactions",
"less_than": "Less than",
Expand Down
2 changes: 1 addition & 1 deletion public/locales/es-ES/translations.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@
"invalid_transaction_hash": "El hash de la transacción es inválido",
"ledger_not_found": "Libro Contable no Encontrado",
"check_ledger_id": "Por favor, comprueba el id de tu libro contable",
"server_ledgers_hint": "Este nodo ({{connection.server.publicKey, truncate(length: 10)}}) solo contiene libros contables {{connection.ledger.validated}}",
"server_ledgers_hint": "Este nodo solo contiene libros contables {{connection.ledger.validated}}",

@pdp2121 pdp2121 Feb 13, 2024

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Also, just a note, if you are unsure about the translations in other languages, you can set them to null and others can fill in later. It would use the default en-US if set to null

"use_search": "Por favor, utiliza nuestra búsqueda",
"ledger_has_no_trans": "Este libro contable no tiene ninguna transacción",
"less_than": "Menos que",
Expand Down
2 changes: 1 addition & 1 deletion public/locales/fr-FR/translations.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,7 @@
"invalid_transaction_hash": "Le hash de la transaction est invalide",
"ledger_not_found": "Registre introuvable",
"check_ledger_id": "Veuillez vérifier votre id de registre",
"server_ledgers_hint": "Ce nœud ({{connection.server.publicKey, truncate(length: 10)}}) ne contient que les registres {{connection.ledger.validated}}.",
"server_ledgers_hint": "Ce nœud ne contient que les registres {{connection.ledger.validated}}.",
"use_search": "Merci d'utiliser notre outil de recherche",
"ledger_has_no_trans": "Ce registre n'a aucune transaction.",
"less_than": "Moins de",
Expand Down
2 changes: 1 addition & 1 deletion public/locales/ko-KR/translations.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,7 @@
"invalid_transaction_hash": "트랜잭션 해시가 유효하지 않습니다",
"ledger_not_found": "원장을 찾을 수 없습니다",
"check_ledger_id": "원장 ID를 확인해주세요",
"server_ledgers_hint": "이 노드 ({{connection.server.publicKey, truncate(length: 10)}})는 {{connection.ledger.validated}} 원장만 포함하고 있습니다",
"server_ledgers_hint": "이 노드 는 {{connection.ledger.validated}} 원장만 포함하고 있습니다",
"use_search": "검색을 이용해 주세요",
"ledger_has_no_trans": "이 원장에는 어떠한 트랜잭션도 없습니다",
"less_than": "미만",
Expand Down