Hi @Newbeeme
As @risible wrote - Remove the last action in the loop - you do not need to increment "for each" loop; it does it automatically. On each iteration, it grabs the next item on the list.
The rest seem ok assuming your links are correct.
What exactly does not work/what happens?
Also, to make your flow faster - do not open and close the browser in each iteration, instead:
1. Open browser before loop
2. In the loop use navigate to URL action
3. Close browser outside of the loop
