-
Notifications
You must be signed in to change notification settings - Fork 94
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
Gj/on chain validation #835
Conversation
## Describe your changes validation for disputes
7cd2a39
to
474bd99
Compare
Use of `Percent` and constant majority value instead of saturated arithmetic.
// You should have received a copy of the GNU General Public License | ||
// along with this program. If not, see <https://www.gnu.org/licenses/>. | ||
|
||
//! Contains common/reusable functionality. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still no changes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Gauthamastro please take a look
* clone of each deposit for store and dispose instead of full deposits vec cloning; * combined logic for deposit processing and fees collected only if there are deposits; * logs levels demoted to debug from info in ocex pallet;
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## Develop #835 +/- ##
===========================================
- Coverage 40.96% 38.73% -2.24%
===========================================
Files 48 47 -1
Lines 2707 2822 +115
===========================================
- Hits 1109 1093 -16
- Misses 1598 1729 +131
☔ View full report in Codecov by Sentry. |
Looks good. Waiting on devops team reply and weights update... |
TODO: Update the parachain port location to 8844 in Thea after testnet testing.