Remove author from Projects

Created on 28 January 2025, 2 months ago

Problem/Motivation

After checking a warning in πŸ› PHP warning Active , I thought about this property and we might not need it at all.
We currently have as author:
- Core modules: hardcoded to "Drupal Core"
- Recipes: empty
- Contrib modules: name of the user that created the module

The person that created a module might or might not be a maintainer of the module, and they might even cancel their account and the module would still exists.

So I'm not sure that author is a useful property right now.

Proposed resolution

Discuss if the above make sense and if it does, clean up code everywhere.

If we want to think about maintainers, that should be a separate follow-up from this issue in my opinion. This should be about clean up a property that is not really useful.

✨ Feature request
Status

Active

Version

2.0

Component

Code

Created by

πŸ‡ͺπŸ‡ΈSpain fjgarlin

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @fjgarlin
  • πŸ‡ΊπŸ‡ΈUnited States chrisfromredfin Portland, Maine

    This is related to πŸ“Œ Cleanup of Project object contract/constructor Active

  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    This runs directly counter to ✨ Add information about authors and maintainers in project details page Active . Which one do we want to do?

  • πŸ‡ΈπŸ‡°Slovakia poker10

    If author != one of the maintainers on d.o., then this is probably not dependent. Otherwise -1 to this. Commented on the related issue.

  • πŸ‡ͺπŸ‡ΈSpain fjgarlin

    I commented on the other issue, I still think that this should be done as author (person that created the node) is not the same as maintainers.

    As mentioned in the issue description

    The person that created a module might or might not be a maintainer of the module, and they might even cancel their account and the module would still exists.

  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    Thanks for clearing that up, @fjgarlin. I think it makes sense, then, to remove the $authors property from Project now, along with its Svelte representation, and add a new $maintainers property in the other issue when the d.o side is ready.

  • Merge request !714Clean up author all over the place. β†’ (Open) created by fjgarlin
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    Closing as a duplicate of πŸ“Œ Cleanup of Project object contract/constructor Active , let's just do this in there.

  • πŸ‡ͺπŸ‡ΈSpain fjgarlin

    Ouch! I had the MR ready, was just waiting for the pipeline to finish.

  • πŸ‡ͺπŸ‡ΈSpain fjgarlin

    But just saw that you already transferred the commit!! Great!

  • Pipeline finished with Success
    about 2 months ago
    Total: 344s
    #417980
  • Pipeline finished with Failed
    about 1 month ago
    Total: 1115s
    #434541
  • Pipeline finished with Success
    about 1 month ago
    Total: 1210s
    #434550
  • Pipeline finished with Failed
    about 1 month ago
    Total: 686s
    #438693
  • Pipeline finished with Failed
    about 1 month ago
    Total: 502s
    #438700
  • Pipeline finished with Failed
    about 1 month ago
    Total: 1418s
    #438708
  • Pipeline finished with Failed
    about 1 month ago
    Total: 1442s
    #438738
  • Pipeline finished with Failed
    26 days ago
    Total: 1291s
    #445788
  • Pipeline finished with Failed
    26 days ago
    Total: 1408s
    #445832
  • Pipeline finished with Failed
    26 days ago
    Total: 1313s
    #445855
  • Pipeline finished with Failed
    26 days ago
    Total: 1458s
    #445866
  • Pipeline finished with Failed
    25 days ago
    Total: 1402s
    #446650
  • Pipeline finished with Success
    24 days ago
    Total: 2749s
    #447405
  • Pipeline finished with Success
    24 days ago
    Total: 1385s
    #447468
  • Pipeline finished with Canceled
    24 days ago
    #447497
  • Pipeline finished with Success
    24 days ago
    Total: 1369s
    #447499
  • Pipeline finished with Success
    24 days ago
    Total: 1470s
    #447566
  • Pipeline finished with Canceled
    24 days ago
    #447856
  • Pipeline finished with Success
    24 days ago
    Total: 1899s
    #447859
  • Pipeline finished with Success
    17 days ago
    Total: 547241s
    #448266
  • Pipeline finished with Failed
    4 days ago
    #463394
  • Pipeline finished with Failed
    4 days ago
    #463463
  • Pipeline finished with Failed
    4 days ago
    #463576
  • Pipeline finished with Success
    3 days ago
    Total: 1597s
    #464402
  • Pipeline finished with Canceled
    3 days ago
    Total: 2201s
    #464496
  • Pipeline finished with Success
    3 days ago
    Total: 2544s
    #464545
Production build 0.71.5 2024