►
From YouTube: Show tags on commits view
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).
B
B
Yeah
I'll
assume
stuff
out
of
the
way.
Okay,
so
I'm
doing
a
walkthrough
of
this
Mr.
To
add
these
badges
to
this
commit
sphere.
These
badges,
particularly
the
the
ones
that
contain
the
tag
names
so
I,
guess
the
first
approach.
I
tried
was
something
like
this,
and
this
was
just
figuring
out
how
to
get
pegs
for
a
given
commit
at
all,
and
this
was
just
something
quick
thrown
together
to
unblock
front-end
development.
B
But
then
we
found
that
it
was
just
like
Hamel
stuff,
so
we
didn't
actually
have
that
much
front-end
stuff
to
do
so.
B
It
wasn't
really
necessary
the
problems
with
this
that
we
find
immediately
are
this
thing
is
per
commit
and
we're
calling
this
for
every
commit
on
that
page
and
I
think
that
the
default
page
size
is
like
40.
So,
yes,.
A
Sean
sorry,
quick
question:
aren't
we
already
downloading
these
tags
on
another
page
or
not
in
this
way,.
B
The
oh
specifically
for
this
feature:
we
want
the
tags
that
point
to
a
given
commit
through
the
commute
on
the
other
page
that
I
think
you're,
referring
to
there's
the
the
commits
detail
page
which
is
like
it
just
shows
not
just
tags
but
like
references
so
including
branches
that
contain
that
commit.
So
there's
like
a
potentially
a
huge
list
of
things
for
older
commits
but
anyways.
This
is
like
a
slightly
different
specificity.
I,
don't
know
these
are
more
specific,
so
it
has
to
point
directly
to
the
commit.
B
That's
what
we're
trying
to
show
anyways
in
this
like
naive
approach
that
I've
done
we're
basically
doing
n,
plus
one
requests
to
Italy
for
every
commit
up
to
like
40
per
page,
so
there's
some
pretty
bad
performance
wise.
The
other
thing
was
that
the
partial
that
we
actually
render
these
badges
in
is
potentially
cashed.
I,
don't
actually
know
what
the
status
of
this
feature
of
flag
is.
But
if
we
change
tags
around,
which
is
potentially
a
thing,
it
would
not
what
we
would
be
showing
still
data
in
the.
C
I
do
know
about
that.
That's
one
of
my
feature
Flags.
So
it's
currently
disabled
because
it's
broken,
but
it's
not
really
broken.
It
relies
on
a
suggesting
Banzai.
So,
basically,
once
we
change
the
redactor
step
in
Banzai,
we
can
turn
that
on
and
the
commit
rendering
will
be
much
faster.
B
Okay,
cool,
let's
see
so
yeah.
Those
are
the
two
issues
with
this
first
naive
approach,
so
the
second
or
the
next.
B
The
next
attempt
is
trying
to
address
n
plus
one
through
making
changes
to
the
RPC
that
we're
actually
calling
so
instead
of
calling
additionally,
this
on
top
of
the
fine
commits
our
PC
that
we're
calling
we
actually
just
kind
of
change
find
commits
to
also
optionally
return
tags.
B
So
that's
this
merge
request.
It's
actually
walking
through
two
Mrs
in
this
presentation,
so
I
went
through
this.
It
was
actually
not
as
hard
as
I
thought,
so
just
kind
of
looking
around
through
the
fine
commits
our
PC.
We
can
actually
see
that
it's
just
you
know,
tacking
things
onto
get
log,
so
for
every
like
option
that
you
send
it,
it
kind
of
just
does
an
extra
little
thing.
So
what
does
that?
Look
like
there's
a
format.
A
B
B
If
we're
like
I
think
the
default
is
just
like
get
the
hash
and
then
the
other
things
are
like.
You
know
some
delimiter
and
then
also
get
the
tags.
That's
basically
what
I
added
to
Italy,
and
it
was
not
that
bad
for
my
first
golang
Mr,
but
you
can
also
see
in
that
same
Mr.
You
can
see
the
goodalee
team,
just
kind
of
puts
me
on
their
back
and
Carries
me
to
the
finish
line.
So
it
was.
It
was
interesting
to
learn.
B
The
Italy
team
is
a
pretty
alright
Bunch
anyways,
so
going
back
to
rails
once
we
actually
got
that
merged
had
to
ask
them
to
publish
the
new
geta
Lee
gem
to
use
on
the
rail
side,
and
that
was
so.
A
B
A
B
A
B
A
B
Because
the
request
signature
has
changed
right.
Okay,
thank
you.
It's
actually
I
think
the
response,
if
you,
if
the
response
includes
like
new
fields
that
aren't
like
that
the
client
doesn't
understand,
I,
think
it
just
kind
of
ignores
them,
so
it's
it's
actually
safe
to
not
have
it
but
having.
It
means
that
we
can
actually
use
those
new
Fields,
okay,
okay,
anyways,
changing
the
RPC
fix
the
M
plus
one
issue
and
then
crashing.
B
Let
me
just
oh
I'll
just
find
it
here,
that's
helper!
Is
it
yeah?
So
this
is
what
the
actual
hash
key
looks
like
and
since
we
load
all
the
tags
for
every
request
to
that
page
so,
like
the
tags
themselves,
won't
be
attached
because
they're
coming
through
the
RPC
anyways,
we
just
do
like
that
calculate
a
check
sum
and
then
we'll
use
that
as
part
of
the
cache
key
and
that
I
guess
that's
kind
of
it.
For
this
Mr
any
questions.
B
D
A
Well,
that
was
really
great
jury
and
I'm
glad
that
your
first
effort
at
modifying
Italy
was
an
enjoyable
one.
I'll
pass
that
feedback
on
to
the
giddly
team
actually,
and
that
was
also
a
great
presentation
if
you
just
want
to
drop
the
bmrs
in
the
in
the
issue
in
our
shared
document.
That
would
be
awesome
and
yeah
thanks
very
much
I'm
going
to
stop
the
recording
unless
anyone
else
has
any
questions
for
Jerry.