fix(manager): stop using state info numblock to calculate batch end height

[srene]

PR Standards

Opening a pull request should be able to meet the following requirements

--

PR naming convention: https://hackmd.io/@nZpxHZ0CT7O5ngTp0TP9mg/HJP_jrm7A

---

Close #1099 , #1092

<-- Briefly describe the content of this pull request -->

For Author:

• Targeted PR against correct branch
• included the correct type prefix in the PR title
• Linked to Github issue with discussion and accepted design
• Targets only one github issue
• Wrote unit and integration tests
• All CI checks have passed
• Added relevant godoc comments

---

For Reviewer:

• confirmed the correct type prefix in the PR title
• Reviewers assigned
• confirmed all author checklist items have been addressed

---

After reviewer approval:

• In case targets main branch, PR should be squashed and merged.
• In case PR targets a release branch, PR should be rebased.

[omritoptix]

I believe we missed handling for num blocks coming from events:

numBlocks, err := strconv.ParseInt(rawEventData.Events["state_update.num_blocks"][0], 10, 64)

also there are more places in the code which seem to refer to num blocks which aren't used (like ValidateBasic of msgUpdateState)

[omritoptix]

Suggested change

	// We're populating the endHeight here with the assumption that state info must contain at least one block and that BDs are in ascending order

[srene]

closing pr, since numblocks removal from state_info (what this pr is fixing) will be reverted.