Sablier updates

[PaulRBerg]

Made a few changes to the Sablier adapters.

• Renamed sablier-v2 to sablier-lockup
• Renamed sablier to sablier-legacy

[PaulRBerg]

Can you please merge this PR at the same with DefiLlama/defillama-server#8909, to avoid any downtime?

[g1nt0ki]

it is hard to tell, is there any logic change other than moving around code & renaming folders?

We will change the name, but renaming folders actually break our system

[PaulRBerg]

Hey @g1nt0ki thanks for merging the PRs/.

Is there any logic change other than moving around code & renaming folders?

No, but as per the other PR (DefiLlama/defillama-server#8909 (comment)), we've renamed our product. Renaming the folders would keep them in sync with the new product names.

If it's too much of a hassle, happy to stick with sablier and sablier-v2, though in that case, I'd like to add a comment in the JavaScript code to explain the folder names.

I see that the following URLs all return 404 now:

• https://defillama.com/protocol/sablier-v1
• https://defillama.com/protocol/sablier-legacy
• https://defillama.com/protocol/sablier-v2
• https://defillama.com/protocol/sablier-lockup

[g1nt0ki]

hi @PaulRBerg sorry, missed the comment about PR in server repo, give it a few hours. Feel like this is a bug with new listings (or renaming protocol in your case), it should fix itself

[PaulRBerg]

Alright, thank you!

[PaulRBerg]

Hey @g1nt0ki, I see that the /sablier-legacy and the /sablier-lockup links work correctly. Thanks very much for that!

However, I am getting 404 not found errors for the /sablier-v1 and /sablier-v2 links.

Would it be possible to set redirects from /sablier-v1 to /sablier-legacy, and from to /sablier-v2 to sablier-lockup, pretty please?

[g1nt0ki]

Hey @g1nt0ki, I see that the /sablier-legacy and the /sablier-lockup links work correctly. Thanks very much for that!

However, I am getting 404 not found errors for the /sablier-v1 and /sablier-v2 links.

Would it be possible to set redirects from /sablier-v1 to /sablier-legacy, and from to /sablier-v2 to sablier-lockup, pretty please?

hi, we should add a redirect, unfortunately we dont have that setup atm.

[PaulRBerg]

Alright, thanks for confirming @g1nt0ki — opened a feature request here: #13396