search

comit-network / ambrosia

UI for trading in the COMIT network
Apache License 2.0
4 stars 3 forks source link

Add sentry #144

Closed da-kami closed 3 years ago

da-kami commented 3 years ago

The first commit removes the About and Wallet page because we actually don't use it. If we decide to want it we can bring it back. At the moment I am going more with an application approach of adding help related features to the Help menu.

Should we add additional contact possibilities in the Help menu?

D4nte commented 3 years ago

The first commit removes the About and Wallet page because we actually don't use it. If we decide to want it we can bring it back.

We have a todo in the current cycle to add content to the About page.

da-kami commented 3 years ago

Please add the usage of sentry as an entry in the changelog. Also please correct the text, let's not collect more data than we need.

There is no changelog on Ambrosia. I am planning a release, I would sum it up manually once the release exists. I don't think we have to over-engineer that yet :)

D4nte commented 3 years ago

Please add the usage of sentry as an entry in the changelog. Also please correct the text, let's not collect more data than we need.

There is no changelog on Ambrosia. I am planning a release, I would sum it up manually once the release exists. I don't think we have to over-engineer that yet :)

I think it'd be good to have a changelog file without over-engineering the release process.

I am happy to add the release file for you :)

da-kami commented 3 years ago

Please add the usage of sentry as an entry in the changelog. Also please correct the text, let's not collect more data than we need.

There is no changelog on Ambrosia. I am planning a release, I would sum it up manually once the release exists. I don't think we have to over-engineer that yet :)

I think it'd be good to have a changelog file without over-engineering the release process.

I am happy to add the release file for you :)

Sure, feel free to do so.

mergify[bot] commented 3 years ago

bors r+

bors[bot] commented 3 years ago

:clock1: Waiting for PR status (Github check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set.

da-kami commented 3 years ago

bors retry

bors[bot] commented 3 years ago

:clock1: Waiting for PR status (Github check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set.

D4nte commented 3 years ago

So what happened? I don't see any mention of automated sentry for the user.