Design feedback wise, looks pretty reasonable given the components you've already got. Looks like you've placed the limit switches in an interesting position though - it they don't trigger they're going to get crushed :) I'd put them at a right angle to the direction of travel. You'll also want to be sure that they will trigger on the aluminium (they should - but with a significantly reduced sensing distance), or you'll want a ferrous metal target.