Does it work now?
You have to restrict the updates, that happen, so it doesn‘t do an endless loop of updating. It’s like a door, where only one is going to be open at a time. It‘s like these doors from the zoo so the tigers don’t escape. Build some working D Latches and use them as caches, so the inputs don‘t immediatly update when the output changes. And when the input changes to the output don‘t update the output until the input doesn‘t change anymore. This way you react to an edge only once. Try building it this way. If you need help with the D Latches tell me. By chaining these together you should be able to count up.