UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Bull 1972 - Launchboard service status visual

Closed Hunter Suchyj requested to merge BULL-1972 into master
6 unresolved threads

Backend changes for service statuses component

Merge request reports

Requires 1 approval from eligible users and Code Owners.

Closed by BRENDAN DERRBRENDAN DERR 1 year ago (Mar 24, 2023 5:00pm UTC)

Merge details

  • The changes were not merged into master.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
1 INSERT INTO services (name, status, description, url, image, created_at, updated_at)
  • 1 INSERT INTO services (name, status, description, url, image, created_at, updated_at)
    2 VALUES ("confluence","Down", "data", "data", "data", now(), now()),
    3 ("confluence","Error", "data", "data", "data", now(), now()),
    • Interesting approach, I would have created some script or some automation to generate these different permutations of parameters for inserting into the db. I see nothing wrong with this since this is for testing but it could get fat fingered and have an error.

    • sequelize has this feature built-in. see the sequelize docs for seeds

    • I wanted to automate it but I figured the real data might be totally different so didn't want to spend much time on it. I also couldnt figure out how to populate the db from the compose or other script

    • Please register or sign in to reply
  • 27 39 ],
    28 40 async (req, res, next) => {
    29 41 try {
    30 const { name, description, url, image } = req.body;
    42 const { name, status, description, url, image } = req.body;
  • 19 19 },
    20 20 ];
    21 21
    22 const getServiceStatusesApi = [
  • 20 20 unique: 'compositeIndex',
    21 21 allowNull: false,
    22 22 },
    23 status: {
  • closed

  • Please register or sign in to reply
    Loading