Skip to content

Remake Twitter handle saving with Vue and modularize account property components - #33217

Merged
PVince81 merged 2 commits into
masterfrom
enh/27869/twitter
Aug 25, 2022
Merged

Remake Twitter handle saving with Vue and modularize account property components#33217
PVince81 merged 2 commits into
masterfrom
enh/27869/twitter

Conversation

@Pytal

@PytalPytal commented Jul 13, 2022

Copy link
Copy Markdown
Member

Remake Twitter handle saving with Vue for accessibility purposes alongside preparations for allowing simple replacement of other account property sections with Vue

Incorporates improvements to accessibility from the https://github.com/nextcloud/nextcloud-vue library over the legacy PHP implementation

Deprecation

The lookupServerUploadEnabled block

<?phpif ($_['lookupServerUploadEnabled']) { ?>
<div class="verify <?phpif ($_['twitter'] === '' || $_['twitterScope'] !== 'public') {
p('hidden');
} ?>">
<img id="verify-twitter" title="<?phpp($_['twitterMessage']); ?>" data-status="<?phpp($_['twitterVerification']) ?>" src="
<?php
switch ($_['twitterVerification']) {
case \OC\Accounts\AccountManager::VERIFICATION_IN_PROGRESS:
p(image_path('core', 'actions/verifying.svg'));
break;
case \OC\Accounts\AccountManager::VERIFIED:
p(image_path('core', 'actions/verified.svg'));
break;
default:
p(image_path('core', 'actions/verify.svg'));
}
?>" <?phpif ($_['twitterVerification'] === \OC\Accounts\AccountManager::VERIFICATION_IN_PROGRESS || $_['twitterVerification'] === \OC\Accounts\AccountManager::NOT_VERIFIED) {
print_unescaped(' class="verify-action"');
} ?>>
<div class="verification-dialog popovermenu bubble menu">
<div class="verification-dialog-content">
<p class="explainVerification"></p>
<p class="verificationCode"></p>
<p><?phpp($l->t('It can take up to 24 hours before the account is displayed as verified.')); ?></p>
</div>
</div>
</div>
<?php } ?>
is deprecated and therefore not brought over as it is only displayed under the condition that your Twitter handle is set and the scope is public which is never the case as the public scope was deprecated in 21.0.1
/**
* Contact details visible on trusted federated servers and in the public lookup server.
*
* @deprecated 21.0.1
*/
publicconstVISIBILITY_PUBLIC = 'public';

Requires

@PytalPytal added this to the Nextcloud 25 milestone Jul 13, 2022
@PytalPytal self-assigned this Jul 13, 2022
@PytalPytal mentioned this pull request Jul 13, 2022
11 tasks
Comment threadapps/settings/src/components/PersonalInfo/TwitterSection/TwitterSection.vue Outdated
@Pytal
Pytalforce-pushed the enh/27869/twitter branch from 76b54fb to b995f76CompareJuly 14, 2022 01:15
@github-actions

This comment was marked as outdated.

@Pytal
Pytalforce-pushed the enh/27869/consolidate branch from 64584ef to a70eb01CompareJuly 15, 2022 03:32
@Pytal
Pytalforce-pushed the enh/27869/twitter branch 2 times, most recently from 0219f8b to a556c67CompareJuly 15, 2022 04:30
@Pytal
Pytalforce-pushed the enh/27869/consolidate branch from a70eb01 to ae18cddCompareJuly 15, 2022 19:15
@Pytal
Pytalforce-pushed the enh/27869/twitter branch 3 times, most recently from 1bb7323 to f644707CompareJuly 15, 2022 20:22
@Pytal

Pytal commented Jul 15, 2022

Copy link
Copy Markdown
MemberAuthor

Keeping as draft pending merge of

@PytalPytal added 3. to review Waiting for reviews feature: accessibility and removed 2. developing Work in progress labels Jul 15, 2022
@Pytal
Pytalforce-pushed the enh/27869/twitter branch 2 times, most recently from 4dbe41e to 6cf836aCompareJuly 15, 2022 20:50
@github-actions

This comment was marked as outdated.

@Pytal
Pytalforce-pushed the enh/27869/twitter branch from 6cf836a to 446b83fCompareJuly 19, 2022 00:34
@Pytal
Pytalforce-pushed the enh/27869/consolidate branch from ae18cdd to 3430808CompareJuly 21, 2022 01:14
@Pytal
Pytalforce-pushed the enh/27869/twitter branch 2 times, most recently from edbce9e to 77f73f8CompareJuly 26, 2022 00:51
@github-actions

This comment was marked as outdated.

Base automatically changed from enh/27869/consolidate to masterJuly 26, 2022 07:13

@artongeartonge left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice cleanup :)

Comment threadapps/settings/src/components/PersonalInfo/shared/AccountPropertyInput.vue Outdated
Comment threadapps/settings/src/components/PersonalInfo/shared/AccountPropertySection.vue Outdated
@Pytal
Pytalforce-pushed the enh/27869/twitter branch from 91fc449 to c6361daCompareJuly 29, 2022 00:13
@Pytal
Pytal requested a review from artongeJuly 29, 2022 00:14
@Pytal
Pytalforce-pushed the enh/27869/twitter branch 2 times, most recently from 5e6458f to db89be3CompareJuly 29, 2022 00:40
@Pytal
Pytalforce-pushed the enh/27869/twitter branch 3 times, most recently from 7e0cd45 to 7fd6634CompareJuly 29, 2022 20:16
@Pytal
Pytalforce-pushed the enh/27869/twitter branch from 7fd6634 to df757f3CompareAugust 9, 2022 03:09
@Pytal

Pytal commented Aug 9, 2022

Copy link
Copy Markdown
MemberAuthor

Rebased and review-ready

@PVince81

Copy link
Copy Markdown
Member

@Pytal this PR seems to contain a lot more changes than just the Twitter handle, in case it's not a mistake, please adjust the PR title and description

@PytalPytal changed the title Remake Twitter handle saving with VueRemake Twitter handle saving with Vue and modularize account property componentsAug 11, 2022
@Pytal

Copy link
Copy Markdown
MemberAuthor

@Pytal this PR seems to contain a lot more changes than just the Twitter handle, in case it's not a mistake, please adjust the PR title and description

Title updated, description already adjusted and commits already split between modularization and Twitter section

This was referenced Aug 12, 2022
@blizzzblizzz mentioned this pull request Aug 24, 2022
@PVince81

Copy link
Copy Markdown
Member

@Pytal rebase needed and also a second review

Signed-off-by: Christopher Ng <chrng8@gmail.com>
Signed-off-by: Christopher Ng <chrng8@gmail.com>

@PVince81PVince81 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@PVince81
PVince81 merged commit 38d0419 into masterAug 25, 2022
@delete-merged-branch
delete-merged-branchBot deleted the enh/27869/twitter branch August 25, 2022 21:40
@blizzzblizzz mentioned this pull request Aug 30, 2022
Sign up for freeto 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.

4 participants

@Pytal@PVince81@tcitworld@artonge