You need to include quarter in your composite keys, I would say, since it's part of the hierarchy. I have just created this dimension as a test and do not see any issues. If the FullDate is not in the hierarchy but is still used against singular years, quarters etc. then it will need to have composite keys too.