Skip to content

Do not show "View Previous App"

Jason Yavorska requested to merge jlenny-master-patch-10216 into master

What does this MR do?

We are currently showing "View previous app" when there is a pending future deployment. The current app is not previous, though, it's just still the current one until the future one is deployed, so this is very confusing. To state it another way, you don't live in your previous house and plan to move to your current one; you live in your current house and plan to move to your next/future one.

This retains the logic to potentially describe them differently in the future, but I'm not sure this is necessary or even what the correct terminology would be. Maybe something like View app (note: another deployment is pending) but this doesn't really feel needed either.

Related to changes from #25351 (closed)

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Orit Golowinski

Merge request reports