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

Authx - Mark extension as table #23601

Merged
merged 2 commits into from
May 27, 2022
Merged

Conversation

colemanw
Copy link
Member

Overview

Marks the Authx core extension as "beta"

Before

"alpha"

After

"beta"

Comments

Since we are pushing notices from Afform telling people to install this extension, we should probably at least mark it as beta.

See https://civicrm.stackexchange.com/questions/41784/is-authx-really-alpha

@civibot
Copy link

civibot bot commented May 26, 2022

(Standard links)

@civibot civibot bot added the master label May 26, 2022
@eileenmcnaughton
Copy link
Contributor

hmm - isn't it stable?

@colemanw
Copy link
Member Author

I'm just trying to push things forward somehow. @totten updated this file recently with a description and links but didn't change the dev status. Maybe he had a reason for leaving it alpha or maybe he forgot?

@totten
Copy link
Member

totten commented May 27, 2022

I'm +1 for bumping it to beta or stable. It's been shipping for while - don't see a reason to hold it back.

I'd lean toward toward stable designation.

Maybe he had a reason for leaving it alpha or maybe he forgot?

Ah, e821ccf - sometimes a copy-edit is just a copy-edit. :)

ext/authx/info.xml Outdated Show resolved Hide resolved
@eileenmcnaughton eileenmcnaughton merged commit b2f942c into civicrm:master May 27, 2022
@eileenmcnaughton eileenmcnaughton changed the title Authx - Mark extension as beta Authx - Mark extension as table May 27, 2022
@eileenmcnaughton eileenmcnaughton deleted the authxBeta branch May 27, 2022 06:43
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