Merge pull request #11398 from Turbo87/trustpub-ui

Implement Trusted Publishing user interface

Author: Turbo87

app/adapters/trustpub-github-config.js

@@ -0,0 +1,7 @@
+import ApplicationAdapter from './application';
+
+export default class TrustpubGitHubConfigAdapter extends ApplicationAdapter {
+  pathForType() {
+    return 'trusted_publishing/github_configs';
+  }
+}

app/controllers/crate/settings.js renamed to app/controllers/crate/settings/index.js

@@ -7,6 +7,7 @@ import { task } from 'ember-concurrency';
 
 export default class CrateSettingsController extends Controller {
   @service notifications;
+  @service store;
 
   crate = null;
   username = '';
@@ -64,6 +65,22 @@ export default class CrateSettingsController extends Controller {
       this.notifications.error(message);
     }
   });
+
+  removeConfigTask = task(async config => {
+    try {
+      await config.destroyRecord();
+      this.notifications.success('Trusted Publishing configuration removed successfully');
+    } catch (error) {
+      let message = 'Failed to remove Trusted Publishing configuration';
+
+      let detail = error.errors?.[0]?.detail;
+      if (detail && !detail.startsWith('{')) {
+        message += `: ${detail}`;
+      }
+
+      this.notifications.error(message);
+    }
+  });
 }
 
 function removeOwner(owners, target) {

app/controllers/crate/settings/new-trusted-publisher.js

@@ -0,0 +1,85 @@
+import Controller from '@ember/controller';
+import { action } from '@ember/object';
+import { service } from '@ember/service';
+import { tracked } from '@glimmer/tracking';
+
+import { task } from 'ember-concurrency';
+
+export default class NewTrustedPublisherController extends Controller {
+  @service notifications;
+  @service store;
+  @service router;
+
+  @tracked publisher = 'GitHub';
+  @tracked repositoryOwner = '';
+  @tracked repositoryName = '';
+  @tracked workflowFilename = '';
+  @tracked environment = '';
+  @tracked repositoryOwnerInvalid = false;
+  @tracked repositoryNameInvalid = false;
+  @tracked workflowFilenameInvalid = false;
+
+  get crate() {
+    return this.model.crate;
+  }
+
+  get publishers() {
+    return ['GitHub'];
+  }
+
+  saveConfigTask = task(async () => {
+    if (!this.validate()) return;
+
+    let config = this.store.createRecord('trustpub-github-config', {
+      crate: this.crate,
+      repository_owner: this.repositoryOwner,
+      repository_name: this.repositoryName,
+      workflow_filename: this.workflowFilename,
+      environment: this.environment || null,
+    });
+
+    try {
+      // Save the new config on the backend
+      await config.save();
+
+      this.repositoryOwner = '';
+      this.repositoryName = '';
+      this.workflowFilename = '';
+      this.environment = '';
+
+      // Navigate back to the crate settings page
+      this.notifications.success('Trusted Publishing configuration added successfully');
+      this.router.transitionTo('crate.settings', this.crate.id);
+    } catch (error) {
+      // Notify the user
+      let message = 'An error has occurred while adding the Trusted Publishing configuration';
+
+      let detail = error.errors?.[0]?.detail;
+      if (detail && !detail.startsWith('{')) {
+        message += `: ${detail}`;
+      }
+
+      this.notifications.error(message);
+    }
+  });
+
+  validate() {
+    this.repositoryOwnerInvalid = !this.repositoryOwner;
+    this.repositoryNameInvalid = !this.repositoryName;
+    this.workflowFilenameInvalid = !this.workflowFilename;
+
+    return !this.repositoryOwnerInvalid && !this.repositoryNameInvalid && !this.workflowFilenameInvalid;
+  }
+
+  @action resetRepositoryOwnerValidation() {
+    this.repositoryOwnerInvalid = false;
+  }
+
+  @action resetRepositoryNameValidation() {
+    this.repositoryNameInvalid = false;
+  }
+
+  @action resetWorkflowFilenameValidation() {
+    this.workflowFilenameInvalid = false;
+  }
+}

app/models/trustpub-github-config.js

@@ -0,0 +1,10 @@
+import Model, { attr, belongsTo } from '@ember-data/model';
+
+export default class TrustpubGitHubConfig extends Model {
+  @belongsTo('crate', { async: true, inverse: null }) crate;
+  @attr repository_owner;
+  @attr repository_name;
+  @attr workflow_filename;
+  @attr environment;
+  @attr('date') created_at;
+}

app/router.js

@@ -19,7 +19,9 @@ Router.map(function () {
     this.route('reverse-dependencies', { path: 'reverse_dependencies' });
 
     this.route('owners');
-    this.route('settings');
+    this.route('settings', function () {
+      this.route('new-trusted-publisher');
+    });
     this.route('delete');
 
     // Well-known routes

app/routes/crate/settings.js

@@ -6,9 +6,11 @@ export default class SettingsRoute extends AuthenticatedRoute {
   @service router;
   @service session;
 
-  async afterModel(crate, transition) {
+  async beforeModel(transition) {
+    await super.beforeModel(...arguments);
+
     let user = this.session.currentUser;
-    let owners = await crate.owner_user;
+    let owners = await this.modelFor('crate').owner_user;
     let isOwner = owners.some(owner => owner.id === user.id);
     if (!isOwner) {
       this.router.replaceWith('catch-all', {
@@ -17,10 +19,4 @@ export default class SettingsRoute extends AuthenticatedRoute {
       });
     }
   }
-
-  setupController(controller) {
-    super.setupController(...arguments);
-    let crate = this.modelFor('crate');
-    controller.set('crate', crate);
-  }
 }

app/routes/crate/settings/index.js

@@ -0,0 +1,21 @@
+import Route from '@ember/routing/route';
+import { service } from '@ember/service';
+
+export default class SettingsIndexRoute extends Route {
+  @service store;
+
+  async model() {
+    let crate = this.modelFor('crate');
+
+    let githubConfigs = await this.store.query('trustpub-github-config', { crate: crate.name });
+
+    return { crate, githubConfigs };
+  }
+
+  setupController(controller, { crate, githubConfigs }) {
+    super.setupController(...arguments);
+
+    controller.set('crate', crate);
+    controller.set('githubConfigs', githubConfigs);
+  }
+}

app/routes/crate/settings/new-trusted-publisher.js

@@ -0,0 +1,8 @@
+import Route from '@ember/routing/route';
+
+export default class NewTrustedPublisherRoute extends Route {
+  async model() {
+    let crate = this.modelFor('crate');
+    return { crate };
+  }
+}

app/serializers/trustpub-github-config.js

@@ -0,0 +1,11 @@
+import ApplicationSerializer from './application';
+
+export default class TrustpubGitHubConfigSerializer extends ApplicationSerializer {
+  modelNameFromPayloadKey() {
+    return 'trustpub-github-config';
+  }
+
+  payloadKeyFromModelName() {
+    return 'github_config';
+  }
+}

app/styles/crate/settings.module.css renamed to app/styles/crate/settings/index.module.css

@@ -1,4 +1,4 @@
-.owners-header {
+.owners-header, .trusted-publishing-header {
     display: flex;
     justify-content: space-between;
     align-items: center;
@@ -43,6 +43,30 @@
     }
 }
 
+.trustpub {
+    background-color: light-dark(white, #141413);
+    border-radius: var(--space-3xs);
+    box-shadow: 0 1px 3px light-dark(hsla(51, 90%, 42%, .35), #232321);
+
+    tbody > tr > td {
+        border-top: 1px solid light-dark(hsla(51, 90%, 42%, .25), #232321);
+    }
+
+    th, td {
+        text-align: left;
+        padding: var(--space-s) var(--space-m);
+    }
+
+    .details {
+        font-size: 0.85em;
+        line-height: 1.5;
+    }
+
+    .actions {
+        text-align: right;
+    }
+}
+
 .email-column {
     width: 25%;
     color: var(--main-color-light);