Paths

Table of Contentst

Differential D6492

[keyserver] enable accepting policies feature
ClosedPublic
Actions

Authored by kamil on Feb 1 2023, 7:01 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Apr 7, 10:03 PM

	Unknown Object (File)
	Sat, Apr 6, 9:11 AM

	Unknown Object (File)
	Fri, Apr 5, 12:01 AM

	Unknown Object (File)
	Fri, Apr 5, 12:01 AM

	Unknown Object (File)
	Fri, Apr 5, 12:01 AM

	Unknown Object (File)
	Fri, Apr 5, 12:00 AM

	Unknown Object (File)
	Thu, Apr 4, 11:54 PM

	Unknown Object (File)
	Mon, Mar 18, 10:28 PM

View All 64 Files

Subscribers

Details

Reviewers

atul
tomek
inka

Commits

rCOMM99ec0f13fd04: [keyserver] enable accepting policies feature

Summary

Updating code version because feature is released. Setting date to run script properly.

Test Plan

Test if yarn script dist/scripts/force-policy-acknowledgment.js works properly.
Test if accepting policy works on native

Diff Detail

Repository

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Feb 1 2023, 7:01 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptFeb 1 2023, 7:01 AM

Harbormaster completed remote builds in B15935: Diff 21758.Feb 1 2023, 7:15 AM

kamil mentioned this in D6394: [landing] update Privacy Policy content.Feb 1 2023, 9:39 AM

kamil added a reviewer: inka.Feb 1 2023, 9:42 AM

kamil added inline comments.

keyserver/src/scripts/force-policy-acknowledgment.js
9	As I said here it is possible to release this tomorrow: https://phab.comm.dev/D6394#inline-43367. Concrete time is not needed (shouldn't cause any harm) so just pushing the date to the next day

kamil published this revision for review.Feb 1 2023, 9:43 AM

atul requested changes to this revision.Feb 1 2023, 10:21 AM

atul added inline comments.

keyserver/src/utils/validation-utils.js
166	We should make this 183 and it should be good to land

This revision now requires changes to proceed.Feb 1 2023, 10:21 AM

Requesting review to get the answer

keyserver/src/utils/validation-utils.js
166	Sorry, but could you say why is that? All native changes are included in yesterday's release, represented by this commit. I am not sure why we need to bump the version higher

atul requested changes to this revision.Feb 1 2023, 10:37 AM

atul added inline comments.

keyserver/src/utils/validation-utils.js
166	Ah okay then we should make it 181 which is the first in the "pair" (181 is the public release paired with 182 which is the staff release)

This revision now requires changes to proceed.Feb 1 2023, 10:37 AM

update version to 181

keyserver/src/utils/validation-utils.js
166	that makes sense to me, thanks for catching it!

Looks good! Sorry for the churn

This revision is now accepted and ready to land.Feb 1 2023, 10:48 AM

Harbormaster completed remote builds in B15943: Diff 21767.Feb 1 2023, 10:58 AM

Closed by commit rCOMM99ec0f13fd04: [keyserver] enable accepting policies feature (authored by kamil). · Explain WhyFeb 2 2023, 3:14 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM99ec0f13fd04: [keyserver] enable accepting policies feature.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

responders/

user-responders.js

2 lines

scripts/

force-policy-acknowledgment.js

2 lines

utils/

validation-utils.js

2 lines

Diff 21758

keyserver/src/responders/user-responders.js

Loading...

keyserver/src/scripts/force-policy-acknowledgment.js

Loading...

keyserver/src/utils/validation-utils.js

Loading...