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

Communicating to the user if the server is not available #171

Merged
merged 6 commits into from
May 18, 2023

Conversation

Kledis98
Copy link
Collaborator

This was done by :
1- Disabling the editor.
2-Changing the placeholder text.

@AndiBraimllari
Copy link
Collaborator

After applying the suggested changes I believe the pipeline should go green.
Feel free to push more code and we can discuss again.

@AndiBraimllari
Copy link
Collaborator

Can you @AndersonCeci create an issue for adding tests to such functionality? We can then address it at another point in time.

@AndersonCeci
Copy link
Collaborator

Can you @AndersonCeci create an issue for adding tests to such functionality? We can then address it at another point in time.

Done #172

@AndiBraimllari
Copy link
Collaborator

Nice, the pipeline is green!
Can you run npm run prettier:fix to format the code?
Also, there's a change in angular.json, can you revert that? Unsure if you've made that change manually or if it was done automatically.

@AndiBraimllari
Copy link
Collaborator

LGTM, merging now!

@AndiBraimllari AndiBraimllari merged commit b0e2999 into OpenCovenant:main May 18, 2023
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