Improve ContractEntry construction

[Buckram123]

This PR implements FromStr for ContractEntry.

Checklist

• CI is green.
• Changelog updated.

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Pages

Latest commit:	0a98956
Status:	✅  Deploy successful!
Preview URL:	https://7b7cf105.abstract-docs.pages.dev
Branch Preview URL:	https://update-improve-contract-entr.abstract-docs.pages.dev

View logs

[codecov-commenter]

Codecov Report

Merging #26 (7d97178) into main (17fbee7) will increase coverage by 0.00%.
The diff coverage is 71.42%.

@@           Coverage Diff           @@
##             main      #26   +/-   ##
=======================================
  Coverage   24.09%   24.10%           
=======================================
  Files         266      266           
  Lines       23369    23375    +6     
=======================================
+ Hits         5631     5634    +3     
- Misses      17738    17741    +3     

Files Changed	Coverage Δ
.../abstract-core/src/objects/entry/contract_entry.rs	46.26% <66.66%> (+0.36%)	⬆️
modules/contracts/apps/croncat/src/utils.rs	96.42% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[adairrr]

Lgtm!

[Buckram123]

@adairrr wdyt, do we need to remove TryFrom for UncheckedEntry?

[Buckram123]

@adairrr wdyt, do we need to remove TryFrom for UncheckedEntry?

And if yes, how we can do it harmless

[adairrr]

@adairrr wdyt, do we need to remove TryFrom for UncheckedEntry?

And if yes, how we can do it harmless

Is there a reason to?

[Buckram123]

@adairrr wdyt, do we need to remove TryFrom for UncheckedEntry?

And if yes, how we can do it harmless

Is there a reason to?

Sorry, meant to say remove TryFrom<String<(got trolled by markdown) in favor of TryFrom<&str> because references are always preferable, and we don't benefit from owning a string anyway. In addition can save few bytes in wasms

[Kayanski]

Shouldn't we keep the From implementation ?

[Buckram123]

Shouldn't we keep the From implementation ?

Is there's a reason to support both string and string slice?

[Kayanski]

Not really no

[CyberHoward]

module tests failing, will check after merging this.