►
From YouTube: Backdrop Weekly Dev Meeting November 18, 2021
Description
No description was provided for this meeting.
If this is YOUR meeting, an easy way to fix this is to add a description to your video, wherever mtngs.io found it (probably YouTube).
A
All
right,
hello,
everyone,
it's
november,
18th
2021,
and
this
is
the
weekly
backdrop
developer
meeting.
We
get
together
every
week
to
talk
about
priorities
and
the
latest
things
happening
in
the
backdrop
world.
Before
we
get
started.
We'll
do
some
introductions.
My
name
is
nate
lampton,
I'm
quick
sketch
on
the
internet
and
I'm
in
oakland
california.
B
Sure
my
name
is
jen
lampton
joining
from
oakland
california.
B
That's
it
for
me.
I
guess
greg.
C
D
E
Hi
phil
out
of
wichita
kansas.
I
don't
really
have
much
to
contribute
on
the
dead
side
of
things,
but
just
kind
of
here
and
here
to
be
a
fly
on
the
wall
and
see
what
discussion
happens.
F
Robert
lang,
foot
folder
on
the
internet
coming
to
you
from
altadena
california,
lately
I've
been
working
on
contrib.
C
A
Andy
we're
just
doing
introductions
if
you
want
to
give
an
intro
feel
free,
if
not
we'll,
just
move
on.
C
B
A
G
Hi,
oh
sorry,
I
just
signed
in,
I
didn't
realize
you
were
waiting
for
me
all
right.
Sorry
am
I
introducing
myself.
I
suspect.
Yes,
yes,
oh
andy,
doc
from
the
cayman
islands.
A
long
time
haven't
been
in
these
meetings,
looking
forward
to
contributing
for
a
change.
A
That's
it
awesome,
welcome
andy
all
right
yeah
we
got
to
off
to
a
really
late
start,
so
it's
probably
a
surprise
to
you
that
we're
still
doing
introductions
yeah.
This
won't
be
a
very
long
meeting.
A
Let's
see
jen,
do
you
want
to
do
the
community
section?
It's
populated
in
the
agenda.
A
I
can
do
it
if
you
want
sure.
Okay,
all
right,
welcome
albany
computers,
that's
steve
morehouse!
We
know
him
from
backdrop.
Live
fame,
he's
our
resident
video
wizard
he's
applied
to
the
contrip
queue
and
has
ported
the
views,
grid,
repeat
module
thanks
and
welcome.
A
Not
that's
really
welcome,
but
welcome
steve
to
the
contrib
group
and
thanks
gregory
for
for
being
our
our
moderator
there,
keeping
an
eye
on
those
requests.
You
kind
of
walked
steve
through
the
process
and
that's
really
excellent
to
grow
our
community.
A
Let's
see,
we
also
have
four
new
contrib
projects
that
were
released
this
week.
Views
json
index
now
repeating
dates
and
uber
cart
feeds.
So
that's
very
exciting.
Four
new
contrib
modules,
all
just
in
the
last
seven
days,
can.
B
I
just
interrupt
and
say
thank
you
indigozella
for
taking
over
views
jason.
I
really
appreciate
help
on
that.
A
B
Awesome
I
created
it
a
long
time
ago.
I
think,
for
the
survey
website
to
pull
in.
B
I
don't
know,
I
don't
remember
what
I
created
for,
but
we
used
it
somewhere
and
I
never
made
a
release
and
solid
noticed
that
it
wasn't
out
and
took
it
over
and
finished
it.
So
thanks.
A
Yeah,
that's
actually
what
powers
the
the
news
feed
on
the
dashboard
so
backdrop
cms.org
runs
it
okay,
well,
that's
kind
of
it
for
the
contrib
updates.
Let's
see,
I
don't
think
that
we
have
specific
topics
for
the
that's
funny
for
for
the
next
release.
Unless
anybody
has
anything
they
brought,
anybody
have
any
things
in
specific
today:
okay,
I'd
like.
C
A
A
It
okay.
Thank
you.
B
I
did
set
up
using
scheduler
module
to
automatically
publish
a
new
post
every
week
until
january
1st,
so
it
will
continue
to
be
there
if
people
want
to
use
it,
but
I
think
maybe
without
the
activity
being
like
hey
someone
go,
make
a
new
forum
post.
It's
not
getting
as
much
attention
as
it
did
before.
So
maybe
the
scheduler
wasn't
there.
A
Yeah,
okay,
okay!
Well,
let's
see
other
than
that.
The
big
topic
of
conversation
right
now
is
that
we're
in
the
midst
of
cutting
a
new
minor
release,
our
new
bug
fix
release
like
right
now.
Actually,
laryn
is
working
asynchronously
with
art,
with
those
of
us
on
the
call
and
has
already
cut
the
tag
for
the
next
version
of
backdrop,
which
will
be
1.20.2,
and
this
is
a
security
release.
A
So
there's
also
going
to
be:
let's
see
what
is
a
119.5
release
that
will
come
out
today
as
well.
It's
not
a
big
security
fix.
It's
just.
An
update
to
ck
editor,
seek
editor,
released,
a
security
update
yesterday
and
we're
just
keeping
pace
to
get
out
that
release.
We
tried
to
have
it
out
yesterday,
but
the
everything
came
out
kind
of
late
in
the
day
yesterday
and
then
we
didn't
have
the
time
to
get
our
code
tested
and
pushed
out.
So
sorry.
A
This
is
a
little
bit
out
of
the
ordinary
to
publish
a
security
release
on
thursday,
but
it'll
be
out
shortly.
A
Let's
see
so
that's
that's
the
big
thing,
also
just
a
as
an
aside
a
nice
thing
about
the
new
version
of
ck
editor.
They,
since
it
kind
of
plays
on
the
accessibility
conversation
from
the
last
meeting.
They
fixed
some
of
the
keyboard
shortcuts,
so
that
now,
when
you're
editing
an
image,
you
can
get
after
the
image
and
add
a
new
line
of
text
or
get
out
of
the
image
to
get
in
between
two
images
that
are
next
to
each
other.
A
I
that
always
drove
me
nuts
that
it
was
really
difficult
to
get
outside
of
editing
an
image
to
get
the
cursor
to
place
immediately
before
or
after
an
image,
and
now
there's
just
some
keyboard
shortcuts,
where
you
can
hit
shift
enter
to
like
get
outside
of
the
image
to
add
a
new
line
afterwards.
So
I
was
really
happy
to
see
those
accessibility,
improvements
to
ck
editor
itself
and
that's
in
the
new
version
that
will
be
in
1.20.2.
A
Okay,
let's
see
the
other
item
that
we've
got
in
the
immediate
horizon
for
1.20.2
is
I'm
sorry.
This
will
actually
be
one
at
20.3.
Is
we're
still
working
on
adding
automated
coding
standards?
Phpcs
checks
it
within
github
to
make
it
so
that
as
pull
requests
are
filed,
they'll
get
automatically
linted
for
coding
standards.
That
issue
is
52
45..
A
A
We
can
make
a
new
repository
that
doesn't
have
any
coding
standards
in
it
yet,
except
for
bare
bones:
minimus
minimal
stuff
that
uses
the
latest
version
of
phpcs
or
we
could
work
on
porting
all
of
the
phpcs
rules
to
the
new
version,
but
that
could
take
some
time
and
in
the
the
queue
indigo
zelda
leans
more
towards
one
or
two,
because
we
can
get
those
things
done
much
more
quickly.
A
Video
bw
panda
and
myself
also
lean
towards
option
number
two
of
just
having
minimal
coding
standards,
checks
that
use
the
latest
version.
So
we
don't
have
to
be
running
something,
that's
old
and
not
supported
and
then
iteratively
like
adding
more
coding
standards,
as
as
we
can
so.
Hopefully
that
will
get
that
issue
moving
forward,
because
even
just
like
the
bare
bones,
minimum
stuff
of
tabs
versus
spaces
would
be
like
just
a
huge
win.
A
A
Okay,
here
we
go,
1.282
looks
like
it.
Actually,
it
contains
21
fixed
issues,
including
the
update
to
ck
editor
4.17.1.
A
A
Oh
and
we,
we
included
that
the
major
performance
improvement
on
the
modules
page
that
that
that's
really
exciting,
which
is
funny
because
our
modules
page
already
loads
pretty
quickly
but
compared
to
some
other
cms
competitors,
but
now
it
loads
really
quickly.
So
it's
very
exciting.
We
had
been
doing
like
a
full
directory
scan,
I
think,
on
every
module
in
the
site.
A
A
Phil,
I
saw
you
unmuted.
Is
there
anything
you
wanted
to
add
there.
A
E
F
C
C
B
I
wanted
to
add
something:
we
also
included
the
flexible,
layouts
ui
change.
That
makes
it
look
more,
like
the
other
managed
blocks
page.
Oh
right,.
A
Yeah,
it's
been
kind
of
a
while,
I
guess
since
1.20.1
came
out
so
there's
actually
a
good
good
chunk
of
fixes
in
here.
A
Okay,
well,
I
don't
think
I
have
anything
else
for
the
next
bug
fix
release
that
we
really
want
to
prioritize.
So
we
can
go
ahead
and
start
talking
about
the
next
minor
version,
1.21
that
will
be
coming
out
early
next
year
january
15th
and
we
have
a
small
number
of
tickets
so
far
that
are
already
slated
that
are
kind
of
underway.
A
I
think
that
there's
no
no
official
advocate
but
herbduel
is
the
person
that
is
pushing
this.
One
forward
might
also
overlap
a
little
bit
with
jen,
because
I
know
she's
working
on
the
flexible
layout
templates
for
other
things.
So,
but
this
it's
a
an
enhancement
that
just
lets
you
add
classes
to
your
regions
when
you're
building
a
custom
layout,
which
is
seems
like
it
would
be
extremely
helpful
for
people
that
are
using
the
custom
layout
builder.
A
That
issue
is
50
44..
Currently
it
needs
some
more
reviewing
and
feedback,
but
it
also
is
slated
as
it
needs
a
little
bit
of
work.
So
it's
it's
ready
to
be
tested
and
more
feedback
can
be
given,
even
if
it
still
needs
a
little
work.
A
Okay,
let's
see
next
up
is
a
simple
fix
or
simple
enhancement.
Issue
1975.,
allow
display
settings,
add
display
settings
for
the
book
navigation
books
put
things
like
the
you
know,
previous
page
next
page
up
to
the
parent
chapter
links
and
this
issue
at
1976
makes
it
so
that
the
the
book
navigation
could
be
moved
above
and
below
other
fields
on
the
node
page.
A
All
right
we've
got
providing
a
new
providing
a
new
hidden
path,
content
type
out
of
the
box
issue,
4903.
A
Let's
see
tim
is
the
advocate
for
this
one.
I
wish
he
was
still
online
for
it,
but
there's
been
a
lot
of
conversation
about
this
one,
because
it's
adding
a
new
content
type
called
card,
and
then
it
also
starts
impacting
the
display
of
the
home
page
to
use
these
new
cards.
A
I
see
that
actually
a
lot
of
people
on
this
call
have
been
participating
in
that
issue.
Jen,
greg
and
phil.
Is
there
any?
Can
one
of
you
summarize
the
state
of
that
issue.
A
The
hidden
hidden
path,
content,
type,
the
card
content
type
and
the
way
what
impacts
it
has.
C
Yeah,
so
I
can
do
you
want
me
to
quickly
summarize
why
we're
trying
to
add
it
or
where
we're
at
or
both?
Why
don't
you
do
both?
So
tim
is
doing
most
of
the
work,
so
we
are
trying
to
see
what
would
be
a
better
fit.
So
the
the
goal
is
to
showcase
to
have
something
in
core
that
actually
showcases
the
page
list
nodes
in
a
way
that
people
understand
them,
but
from
the
mock-ups,
and
that
the
tests
that
we've
been
doing.
C
We
also
thought
that
it
would
provide
the
default
out-of-the-box
experience
with
some
more
content
that
makes
this
the
site
look
like
a
site.
So
that's
a
that's
a
you
know,
happy
side
effect
of
the
of
the
work
that
we've
been
doing
now,
we're
sort
of
like
we're
not
stuck
like.
We
know
how
to
go
forward
from
that.
It's
just
that.
We
need
to
make
some
decisions.
C
What
to
call
these
things.
What's
the
best
implementation,
so
yeah,
I
I
have
very
high
confidence
that
this
will
get
in
for
121..
We
just
need
to
iron
out
details.
I
guess.
A
Yeah,
that
is,
since
I
haven't
read
over
it.
This
is
probably
something
that's
already
been
brought
up,
but
the
mock-ups
I've
seen
put
three
cards
on
the
homepage.
A
very
typical.
You
know,
like
hey
here's
three
pieces
of
content
that
you
might
see
like
in,
like
a
recommended
content
or
see
more
or
something
like
that.
A
A
very
common
use
case
is:
are
we
planning
on
doing
those
three
cards
as
like
as
a
view
you
know,
in
which
case
it
would
be
a
single
column,
even
though
there's
three
cards
in
it
or
three
columns
somehow
like
manually
placed
in
there,
which
would
also
be
possible
out
of
the
box.
C
C
Although
at
some
point
it
was
sort
of
like
conflicting
and
saying
we
want
to
showcase
how
you
have
these
things
as
a
group,
but
we
already
cannot
like.
Why
would
we
showcase
page
less
nodes
as
part
of
views
when
we
can
do
that
with
non-page
lists
like
with
actual
normal
nodes,
but
having
them
being
the
like
at
the
homepage
and
actually
the
content?
The
text
content
of
its
card
basically
explains
what
this
feature
is,
so
it
might
be
mock-up,
but
it's
also
useful.
A
C
Actually,
the
the
the
titles
of
the
nodes
are
linking
to
the
nodes,
but
because
the
pageless
nodes,
if
you're
a
person
that
doesn't
have
the
permission
to
view
page
list
nodes,
you
go
get
a
404,
but
that's
something
that
we
can
sort
out.
Basically,
don't
don't
render
the
the
titles
as
links.
That's
that's
the
solution.
So.
C
C
In
a
name,
yeah,
there's
there's
more
and
more
serious
situation
of
stack
like
in
my
head,
which
is
like
no
way
to
go
forward
because
it's
like
there's
no
solution.
So
we
have,
as
we
have
multiple
solutions.
We
just
need
to
decide
or
suggestions
and
we
need
to
decide
which
one
to
go
with
so
yeah.
G
C
Of
the
cards
and
the
styling,
I
guess
and
also
the
thing
that
I
mentioned
with
with
the
title,
so
it's
basically
team
finding
some
time
to
work
on
the
feedback
that
we
provided.
D
Okay
and
yeah-
I
think
we
stuck
sort
of
stuck
in
so
that
tim
has
to
polish
his
pr
a
bit
so
that
they
then
have
a
good
basis
for
the
rest
of
the
discussion,
because
I
have,
for
example,
some
ideas,
but
as
long
as
the
pr
is
not
really
ready,
it
doesn't
make
sense
to
discuss
it
in
my
opinion,
so,
and
he
has
said
that
he
is
wants
to
work
on
it.
So
I'm
waiting
for
this.
A
We
have
some
follow-up
items
from
that:
telemetry
module
is
in
the
1.20
release
of
backdrop,
there's
a
couple
dozen
backdrop
sites
that
are
reporting
data
back
to
backdropcms.org,
which
is
great
to
see
that
information
coming
in,
but
right
now
it's
a
purely
opt-in
system
where
it's
on
it's
in
backdrop
sites,
but
you
have
to
turn
it
on
and
even
new
existing
sites
don't
have
it
turned
on,
and
the
issue
5253
is
the
issue
to
discuss
how
we
can
get
that
site
or
how
we
get
telemetry
turned
on
on
new
sites,
so
yeah
that
issue
is
really
just
kind
of
getting
started.
A
It's
actually.
This
is
this
kind
of
still
a
little
bit
unclear
as
to
the
the
way
in
which
we
managed
to
do
this.
I
think
last
meeting
we
had
discussed
that
that
this
is
a
actually
kind
of
like
a
policy
problem
that
the
pmc
should
probably
discuss
about
opt-in
versus
opt-out
settings
just
in
general,
because
it
would
apply
to
multiple
things.
A
I
don't
think
that
tim
I
mean
tim
was
the
one
that
suggested
that
we
take
it
to
the
pmc.
Last
meeting
and
I'm
not
sure
if
he
did
quite
yet
but
yeah
seems
like
that's
a
good
example
of
something
that
should
be
should
be
decided
centrally.
B
A
Okay,
let's
see-
and
lastly,
I
haven't
put
myself
down
as
an
advocate
for
this
one,
I'm
already
really
trying
to
prioritize
automatic
updates
and
a
lot
of
the
work
that
has
been
started
but
not
finished.
There.
Things
like
the
code
signing
work
and
the
pre-flight
checks,
like
there's
lots
of
things
in
automatic
updates
that
I've
been
kind
of
hoping
to
to
move
forward.
A
But
another
issue
that
is
weighing
on
me
right
now
is
issue
4122,
which
is
the
ck
editor
issue:
security,
five
and
the
next
step.
There
is
making
a
contrib
module,
probably
for
ck
editor
five.
A
That
would
be
slated
as
like
a
a
core
target
module
so
that
we
can
do
the
development
and
contrib
do
all
the
testing
there
and
then,
when
it
seems
ready
to
move
past
proof
of
concept,
we
can
consider
putting
it
into
core
and
then
maybe
optional
first
tell
people,
maybe
they
should
upgrade
and
then
maybe
out
of
box
enabled
later
replacing
the
current
c
editor
module,
which
would
be
kind
of
slated
just
for
c
c
editor.
A
Four,
the
new
one
will
be
just
seek
editor
five
because
they
require
different
integration
plugins
for
each
version,
so
we
may
have
a
transitionary
period
where
we
have
both
enabled
at
the
same
time,
I
don't
think
everybody
was
here
last
meeting
where
and
mentioned
that
in
the
past,
like
seek
editor
5
had
this
giant
red
flag
for
us
that
it
wasn't
content
compatible
with
previous
content.
So
if
you
edited
content
from
before,
you
started
using
seek
editor
5,
it
might
literally
remove
or
delete
content
out
of
your
text
area.
A
So
just
by
hitting
edit
not
doing
anything
and
then
hitting
save
ck
editor
might
actually
actually
wipe
out
some
of
the
previous
content
there
if
it
didn't
match
the
markup
standards
that
it
requires,
but
the
new
version
of
cq
or
that
they've
now
done
some
work
to
make
a
plug-in
where,
if
seacater
encounters
incompatible
html
with
its
new
dom
structure
that
it
just
locks
that
content
and
makes
it
non-editable,
except
through
like
the
source
mode.
A
G
A
G
That's
the
question,
though,
is
so
it's
possible
now
for
us
to-
I
guess
so
it's
possible,
but
it
would
be
reasonable,
reasonably
simple
or
doable
for
us
to
actually
have
an
upgrade
to
ck
editor
5.
A
Yes,
I
think
so
the
the
upgrade
itself
would
be.
What
I'm
envisioning
at
least,
is
that
the
upgrade
would
really
just
be.
If
you
switch
on
ck
editor
five,
it
copies
your
ck
editor
for
text
format
configuration
so
the
toolbar
configuration
and
matches
the
the
toolbar
setup,
but
other
than
that
like
that
would
be
it
like.
It's
really.
A
It's
just
like
turning
on
a
different
editor
per
text
format
and
copying
your
toolbar
configuration
and
then,
when
you
go
to
edit
your
content,
I
wasn't
planning
on
doing
any
kind
of
like
bulk
update
of
all
of
your
content.
Instead
seek
editor
5
just
when
it
encountered
some
old
content
it
may
or
may
not,
depending
on
the
formatting
of
your
content,
have
certain
sections
that
are
not
editable
within
the
editor.
A
G
Right,
but
is
there
any
absolute
need
in
terms
of
security
or
otherwise
for
us
to
move
to
ck
editor
five.
A
Yeah
sick,
editor,
well,
stick
editor,
4
will
reach
an
end
of
life.
I
think
not
in
2022
but
in
2023.
A
So
there's
not
a
big
hurry,
but
we
obviously
need
to
move
to
something
ultimately
because
it
won't.
It
won't
last
forever.
C
C
So
basically,
could
we
allow
old
content
to
be
edited
via
the
cq
editor
4
and
then
also
provide
new
installations
just
with
ck805
and
then
at
some
point
in
2023,
when,
like
backdrop
2.0,
we
just
remove
tickets
or
fall
together.
Would
that
be
an
acceptable
thing
or
just
do
you
think
too,
messy
and
too
complicated.
A
No,
I
mean
that
would
completely
work.
If
you
wanted
to
to
have
two
text
formats,
one
for
four
and
one
for
five
and
you
could
switch
between
them,
they
could
even
be
configured
exactly
the
same
and
you
would
just
be
literally
talking
toggling
it
on
and
off.
However,
I
don't
think
that
we
could
necessarily
say
this
is
what
all
sites
will
get
that
will
turn
on
a
second
text
format
for
every
existing
text
format.
A
I
think
that
that's
like
that
would
be
a
really
good
option
to
for
site
owners
to
use
to
like
set
up
a
second
text
format,
but
I'm
not
sure
that
we
would
provide
that
as
an
automatic.
Like
all
sites
now
have
double
the
text
formats
you
know
yeah,
but
that
yeah,
that
would
totally
be
a
legitimate
way
of
doing
a
conversion
over
a
long
period
of
time,
if
you're
a
site
owner.
C
Yeah
and
since
we're
not
releasing
bankrupt
2.0
up
until
2025,
then
maybe
and
from
the
moment
that
ck
804
becomes
end
of
life.
Until
then,
it's
a
matter
of
the
site
administrators
sort
of
like
choice,
and
then
we
remove
it
completely
from
like
as
unsupported
or
something.
A
C
A
Yep,
basically,
exactly
that
we
deprecate
seek
editor
for
out
of
box.
New
installs
will
ship
with
c
candidate
five,
because
there's
no
existing
content
to
worry
about
and
that
we
encourage
people
to
move
off
of
four
and
on
to
five
we
might
start
putting
status
report
messages
in
there.
You
know
saying:
hey
you,
gotta
get
off
of
four,
that
sort
of
thing:
yeah.
G
I'm
gonna
vote
that
if
you
find
the
time
nate,
if
you
had
to
pick
one
of
the
two
things
to
deal
with,
I
I
would
prefer
that
you
dealt
with
the
ck
editor
rather
than
the
automatic
upgrades.
I
think
that
that
sounds
more
urgent
to
me.
A
G
From
that,
the
fact
that
I'm
pretty
sure
speaking
for
myself,
but
I'm
sure
most
of
us
at
least
in
this
meeting,
will
agree
that
the
jquery
and
javascript
and
ck
edit
this
stuff
is,
is
worse
than
greek.
Sorry
greg
that
to
the
rest
of
us.
I
I
have
no
clue
what
it's
doing
half
the
time,
so
I
don't
think
most
of
I
don't
think
many
of
us
could
could
really
tackle
that.
Far
from
you
really.
C
Like
if
you
wanna
it's
sort
of
like
a
what
you
call
it
is
that
in
greece
we
don't
say
it
sounds
great
to
us.
It
sounds
chinese
to
us
but
yeah
whatever.
So
I
think.
C
Yeah,
that's
the
equivalent
expression
but
yeah
other
than
that.
What
I
was,
what
I
wanted
to
say
is
that
the
sooner
we
get
ck
editor
5
in
the
less
sites
we'll
have
to
worry
about
moving,
so
the
new
installations
will
start
off
with
security
five,
so
it
would
be
less
of
a
problem.
The
whole
migration
will
move
whatever
yeah.
A
C
A
Well,
I
have
to
have
this
meeting
run
too
short,
but
I've
got
to
end
the
meeting
we're
at
the
top
of
the
hour.
Already
more
meetings
happening,
yeah
go
ahead,
robert
really.
F
Quick
and
it's
probably
a
reminder
to
everyone
else,
but
I
don't
know
how
far
along
does
a
pr
have
to
be
to
go,
get
into
the
milestone
for
a
minor
release
milestone.
I
was
looking
at
the
docs
page
for
it,
but
it
wasn't
clear.
Dude.
A
It
doesn't
have
to
be
anywhere
at
all.
It
just
has
to
have
someone
advocate
for
it.
F
C
Up
a
maybe
we
can
discuss
this
in
the
next
dev
meeting.
I
wanted
to
bring
up
something
very
similar,
so
I
started
writing
tests
for
a
very
small
vr
and
the
tests
ended
up
being
like
80
of
the
work.
So
I
I
wanted
to
switch
the
the
issue
to
needs
review,
but
then
indica
zello
was
setting
it
back
back
to
needs
work
because
the
tests
were
not
passing,
whereas
I
wanted
feedback
on
the
current
implementation.
C
So
I
want
us
at
some
point
to
discuss
what
is
the
appropriate
workflow
and
which,
which
labels
we
use
in
order
to
invite
people
to
actually
review
the
code.
Even
though
tests
are
not
passing
because
we
need
so
like
feedback
on
how
to
move
forward
so
yeah.
F
B
Long
as
a
second
person
agrees
that
that
should
be
in
the
milestone,
you
can
add
it,
so
usually
one
person
will
add
miles
to
a
candidate
and
another
person
marks
it
as
in
the
milestone.
As
long
as
it
has
an
advocate,
that's
all
it
needs.
It
doesn't
need
to
work
so.
C
But
I
was
using
the
needs
feedback
for
the
implementation,
not
the
actual
code,
like
I
needed
some
code
review
for
people
to
just
say
yes,
you're
doing
this
right
or
wrong,
or
maybe
I
have
questions
about
my
implementation,
like
code
implementation,
not
the
technical
solution,
which
is
what
I
think
the
needs
feedback
was
for.
So
I
thought
that
we
had
the
needs
feedback
for
what
should
I
call
this
like
the
cards
name
for
the
thing
that
we
were
discussing
before.
C
That's
when
I
set
a
an
issue
to
needs
feedback,
if
I
need
feedback
on
to
name
things,
if
I
need
someone
to
review
my
code
and
give
me
feedback
on
the
code,
I
used
to
say:
needs
review,
correct,
yeah.
We
can
discuss
this
because
it's
going
to
be
it's
going
to
take
longer
we're
over
time
as
well.
So,
okay,
all
right.
A
All
right
well,
thank
you,
everybody
for
joining
the
meeting.
It
was
short
and
sweet.
Thank
you,
everybody
and
we'll
catch.
You
guys
all
online
and
we'll
probably
see
the
release
of
120.2
within
the
next
hour
or
two.
The
tag
is
already
cut.
We
just
need
to
push
the
release
so
that'll
be
exciting.
B
B
Could
use
eyes
on
the
release,
notes
and
the
security
advisory?
If
someone
wants
to
just
look
at
those
and
make
sure
they
look,
okay,.