fix: remove the "DOI" from alternateIdentifiers

It kinda historically dragged through time from the original
1056afb49fd945afc471e200d782bbff9de43cf1 in dandi-cli where it
was added to "identifiers".  As the wise @tmorell has mentioned,
since it is the datacite which is to provide DOI, it would ignore
DOI "alternateIdentifiers".  I think it makes sensse overall, although
there could potentially be multiple DOIs for a single dandiset -- nothing
in DOI principle forbids it. But since original purpose here is not
clear -- we better just strip it away since it should be the DOI
minted by datacite fabrica as the one for the PublishedDandiset

dandischema/datacite/__init__.py

@@ -79,11 +79,6 @@ def to_datacite(
         attributes["event"] = "publish"
 
     attributes["alternateIdentifiers"] = [
-        # TODO: the first element is ignored, not sure how to fix it...
-        {
-            "alternateIdentifier": f"https://doi.org/{meta.doi}",
-            "alternateIdentifierType": "DOI",
-        },
         {
             "alternateIdentifier": f"https://identifiers.org/{meta.id}",
             "alternateIdentifierType": "URL",

dandischema/datacite/tests/test_datacite.py

@@ -451,13 +451,6 @@ def test_datacite_publish(metadata_basic: Dict[str, Any]) -> None:
                 ],
                 "doi": f"10.80507/dandi.{dandi_id_noprefix}/{version}",
                 "alternateIdentifiers": [
-                    {
-                        "alternateIdentifier": (
-                            f"https://doi.org/10.80507"
-                            f"/dandi.{dandi_id_noprefix}/{version}"
-                        ),
-                        "alternateIdentifierType": "DOI",
-                    },
                     {
                         "alternateIdentifier": f"https://identifiers.org/{dandi_id}/{version}",
                         "alternateIdentifierType": "URL",