Skip to content
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

Adding VariantId and VariantName #589

Merged
merged 3 commits into from
May 20, 2024
Merged

Adding VariantId and VariantName #589

merged 3 commits into from
May 20, 2024

Conversation

Emily-ejag
Copy link
Contributor

this pr adds the variant id and variant name, and if the admin has it, then it shows on view params

Copy link

github-actions bot commented May 17, 2024

Visit the preview URL for this PR (updated for commit 570ba44):

https://roar-staging--pr589-enh-add-variant-name-keli2x24.web.app

(expires Mon, 27 May 2024 21:46:26 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 2631e9c58fd0104ecbfddd72a62245ddac467460

Copy link

cypress bot commented May 17, 2024

Passing run #1872 ↗︎

0 26 0 0 Flakiness 0

Details:

Tests for PR 589 "Adding VariantId and VariantName" from commit "570ba4442d27660...
Project: roar-dashboard-e2e Commit: 570ba4442d
Status: Passed Duration: 05:13 💡
Started: May 20, 2024 10:13 PM Ended: May 20, 2024 10:19 PM

Review all test suite changes for PR #589 ↗︎

@Emily-ejag Emily-ejag requested review from richford, kellyel, lucasxsong and ksmontville and removed request for richford May 17, 2024 17:14
src/components/CardAdministration.vue Outdated Show resolved Hide resolved
src/components/CardAdministration.vue Outdated Show resolved Hide resolved
src/components/CardAdministration.vue Outdated Show resolved Hide resolved
@Emily-ejag Emily-ejag requested a review from richford May 18, 2024 00:32
@richford richford merged commit 2eb60ac into main May 20, 2024
16 checks passed
@richford richford deleted the enh/add-variant-name-id branch May 20, 2024 22:47
@Emily-ejag Emily-ejag self-assigned this May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants