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

Add sensor #7

Merged
merged 10 commits into from
Sep 24, 2023
Merged

Add sensor #7

merged 10 commits into from
Sep 24, 2023

Conversation

LeTamanoir
Copy link
Member

@LeTamanoir LeTamanoir commented Sep 21, 2023

No description provided.

@Nfire2103
Copy link
Contributor

On pourrait peut être mettre la dernière version de Solidity sur les contrats

@Nfire2103
Copy link
Contributor

J'ai remarqué que le signe dans l'exemple de l'inéquation n'est pas le même que celui du if

@Nfire2103
Copy link
Contributor

On peut supprimer le dossier librairy aussi, on a plus besoin de Set

@Nfire2103
Copy link
Contributor

Il reste un TestEvent dans au début de l'ExampleImplementation

@Nfire2103
Copy link
Contributor

Dans le postHookSuccess ça appel super.postHookSuccess mais dans l'abstract il y a pas de postHookSucess

@Nfire2103 Nfire2103 closed this Sep 24, 2023
@Nfire2103 Nfire2103 reopened this Sep 24, 2023
@LeTamanoir
Copy link
Member Author

Dans le postHookSuccess ça appel super.postHookSuccess mais dans l'abstract il y a pas de postHookSucess

c'est parce que le postHookSuccess c'est dans le Direct

@LeTamanoir LeTamanoir merged commit 76153be into main Sep 24, 2023
1 check passed
@LeTamanoir LeTamanoir deleted the add-sensor branch September 24, 2023 20:06
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.

2 participants