The datum target indicators look a little bit confusing. A1 and A2 look like they are really on the upper edge. It would not be a subject for rejection of a drawing which is ready for release but as you have to do the CAR-checks anyway please modify like shown below. (Generally it is better to show the side of the part on which you find the datum targets but that is not mandatory.)