cartesi / rollups-contracts

Smart Contracts for Cartesi Rollups
https://cartesi.github.io/rollups-contracts/
Apache License 2.0
17 stars 37 forks source link

refactor!: remove application address relay #228

Closed guidanoli closed 5 months ago

guidanoli commented 6 months ago

Closes #225

ZzzzHui commented 6 months ago

Here in the library InputEncoding, it says such as portals and relays. Now there's only portals. So we should update this as well. Also, now that we've removed address relay, I'm thinking about whether we need to introduce the new term relay at all, or should we keep using the word portal? On the other hand, the word relay still makes sense and is more general. What do you think?

guidanoli commented 6 months ago

Also, now that we've removed address relay, I'm thinking about whether we need to introduce the new term relay at all, or should we keep using the word portal?

I agree. Let's call them portals in another PR.