Sorry, you didn't provide enough information. What is each button supposed to be doing, what steps are you trying to do with the macro, etc.?
Without that, and possibly even more information, we can only tell you that it didn't work because your macro is incorrect or incorrectly assigned to the button.
You can use this as the basis for your macro button (instead of the macro):
Code:
DoCmd.RunCommand acCmdSaveRecord 'saves the record
DoCmd.Close acForm, Me.Name, acSaveNo 'closes the current form
DoCmd.OpenForm "PutYourFormNameYouWantToOpenHere", acNormal 'opens the other form
Where it says "PutYourFormNameYouWantToOpenHere" replace those words with the actual name of the form you want to open. Remember to keep it within the quotation marks (the quotation marks must be included).
Where it says Me.Name don't change it, that code tells it, within the context of the closing of the form to close the form with the name of the current form that this code is on. Me. is a programming shortcut that lets you not have to fully type out the form reference if the code is on the same form as you are referring to.