Skip to content

improved spelling GitHub, upgraded URLs, enabled translation NL#96

Open
PanderMusubi wants to merge 2 commits intodashpay:masterfrom
PanderMusubi:master
Open

improved spelling GitHub, upgraded URLs, enabled translation NL#96
PanderMusubi wants to merge 2 commits intodashpay:masterfrom
PanderMusubi:master

Conversation

@PanderMusubi
Copy link

No description provided.

@PanderMusubi
Copy link
Author

@pshenmic and @owl352 what to do with this failing test?

@pshenmic
Copy link
Collaborator

pshenmic commented Feb 9, 2025

Let us look at that quickly

@PanderMusubi
Copy link
Author

Ah, thanks, that exaplains. Will commit a fix soon

@PanderMusubi
Copy link
Author

I have fixed the README. However, I did not touch the config for the CI / testing on GitHub. @pshenmic If you look at the error in thte log file, it cencerns something else. Can you see that?

@owl352
Copy link

owl352 commented Feb 9, 2025

It looks like dependency issues because if you try to run the tests on the master you get the same result, but a while ago it was running all the tests successfully

@owl352
Copy link

owl352 commented Feb 9, 2025

The results of my micro investigation led me to believe that the timeout errors are due to the fact that after running grunt watch, we get an infinite run with the text Waiting.... I'm confused

@pshenmic
Copy link
Collaborator

pshenmic commented Feb 9, 2025

Fixed in #97

Testnet Insight backend endpoint was changed from 3001 to 80 not a long time ago and E2E tests are relying on it.

Please pull fresh changes from the master and let me know if it works now

@PanderMusubi
Copy link
Author

Thank. Test is now okay.

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