Skip to content

Use more meaningful names for step cards in CreateWikiWizard#1106

Open
outdooracorn wants to merge 5 commits intomainfrom
improve-naming
Open

Use more meaningful names for step cards in CreateWikiWizard#1106
outdooracorn wants to merge 5 commits intomainfrom
improve-naming

Conversation

@outdooracorn
Copy link
Member

Bug: T419207

rosalieper and others added 5 commits March 6, 2026 17:59
* Move `TemporalityProfileEditWizardStep` to where it is used in `Pages/ManageWiki/cards`
* Remove `dissmissable` property from `TemporalityCreateWikiWizardStep` and the optional close-dialog icon that will never be displayed
* Remove `dissmissable` property from `TemporalityProfileEditWizardStep` and the `v-if` logic for the close-dialog icon as that will always be displayed
* Remove unused `buttonText` property from `CreateWiki`
@outdooracorn outdooracorn self-assigned this Mar 19, 2026
@github-actions
Copy link

Deployment previews on netlify for branch refs/pull/1106/merge will be at the following locations (when build is done):

@outdooracorn
Copy link
Member Author

Not sure how I feel about this but thought I'd put it up for others to have a look at. If we like this, we should do the same for ManageWiki/Cards/Profile.vue as well.

otherAudience: ''
},
stepThree: {
temporality: {
Copy link
Contributor

Choose a reason for hiding this comment

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

what do you think of adding the word "step" to this names, so that we do not have to reference it with temporality.temporality.

Copy link
Contributor

Choose a reason for hiding this comment

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

Nice idea I think I like that too. I think specifically this is looking at R135 below

@rosalieper
Copy link
Contributor

Not sure how I feel about this but thought I'd put it up for others to have a look at. If we like this, we should do the same for ManageWiki/Cards/Profile.vue as well.

Yes, I like this! Thank you Ollie

Base automatically changed from T419207 to main March 19, 2026 14:12
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.

3 participants