add previewnet entry into service

[bthaile]

add previewnet entry into service

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
fcl-discovery	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 23, 2024 6:04pm

[chasefleming]

You'll also need to change any mentions of crescendo to previewnet in the codebase.

[bthaile]

You'll also need to change any mentions of crescendo to previewnet in the codebase.

I only found one instance of "crescendo" of which I updated in this PR. Is there other places I need to look outside of this repo?

[chasefleming]

Ok looks like it was already changed to previewnet everwhere else, except the services key wasn't so should be good: https://github.com/onflow/fcl-discovery/blob/master/helpers/constants.js#L8

Have you tested this?

[bthaile]

Yes, tested it locally using flow port. I added a previewnet network configuration to flow port. Not sure if there are cases for flow port to support previewnet, other than me updating scripts and transactions to be cadence v1 compat. I might be good to test locally only.

[chasefleming]

Approved, but one more spot to check might be to make sure nothing needs to be different in the metadata for previewnet: https://github.com/onflow/fcl-discovery/blob/master/data/metadata.json