Skip to content

[dashboard] allow no expiration date PAT #19327

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jan 17, 2024
Merged

[dashboard] allow no expiration date PAT #19327

merged 4 commits into from
Jan 17, 2024

Conversation

mustard-mh
Copy link
Contributor

@mustard-mh mustard-mh commented Jan 16, 2024

Description

This feature only enabled with dataops Feature Flag enable (both API call and UX)

UX Preview (mock 8 days forward), updates: show warning icon if token never expires + tooltip

SCR-20240117-opsz

Related Issue(s)

Relates EXP-1084

How to test

https://hw-forever-pat-2.preview.gitpod-dev.com/workspaces

With Feature Flag OFF

  • Turn off dataops Feature Flag for hw- prefix preview env and your userID
  • There should have no No expiration option when create / edit PATs

With Feature Flag ON

Need to enable dataops Feature Flag with your userID

  • Try CRUD PATs with no expiration option
  • Verify if API calls work with generated no expiration token

✅ Share test results

UX API call
SCR-20240117-opsz SCR-20240117-dlzg

Documentation

Preview status

gitpod:summary

Build Options

Build
  • /werft with-werft
    Run the build with werft instead of GHA
  • leeway-no-cache
  • /werft no-test
    Run Leeway with --dont-test
Publish
  • /werft publish-to-npm
  • /werft publish-to-jb-marketplace
Installer
  • analytics=segment
  • with-dedicated-emulation
  • workspace-feature-flags
    Add desired feature flags to the end of the line above, space separated
Preview Environment / Integration Tests
  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-large-vm
  • /werft with-gce-vm
    If enabled this will create the environment on GCE infra
  • /werft preemptible
    Saves cost. Untick this only if you're really sure you need a non-preemtible machine.
  • with-integration-tests=all
    Valid options are all, workspace, webapp, ide, jetbrains, vscode, ssh. If enabled, with-preview and with-large-vm will be enabled.
  • with-monitoring

/hold

@roboquat roboquat added size/XL and removed size/L labels Jan 16, 2024
@mustard-mh mustard-mh changed the title WIP [dashboard,papi] allow no expiration date PAT [dashboard,papi] allow no expiration date PAT Jan 16, 2024
@mustard-mh mustard-mh marked this pull request as ready for review January 16, 2024 19:23
@mustard-mh mustard-mh requested a review from a team as a code owner January 16, 2024 19:23
@mustard-mh mustard-mh marked this pull request as draft January 17, 2024 07:33
@mustard-mh mustard-mh marked this pull request as ready for review January 17, 2024 08:49
getDate: () => dayjs().add(d, "days").toDate(),
}));

// Max value of timestamp(6) in mysql is 2038-01-19 03:14:17
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️ thanks for checking that

@mustard-mh mustard-mh changed the title [dashboard,papi] allow no expiration date PAT [dashboard] allow no expiration date PAT Jan 17, 2024
@roboquat roboquat merged commit 5ce88c3 into main Jan 17, 2024
@roboquat roboquat deleted the hw/forever-pat-2 branch January 17, 2024 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants