pg.ddx.io pgsql-bugs@postgresql.org mailing list archive
help / color / mirror / Atom feedError in form on site commitfest.postgresql.org
4+ messages / 3 participants
[nested] [flat]
* Error in form on site commitfest.postgresql.org
@ 2025-02-13 12:46 Vladlen Popolitov <v.popolitov@postgrespro.ru>
2025-02-13 14:41 ` Re: Error in form on site commitfest.postgresql.org Daniel Gustafsson <daniel@yesql.se>
0 siblings, 1 reply; 4+ messages in thread
From: Vladlen Popolitov @ 2025-02-13 12:46 UTC (permalink / raw)
To: pgsql-bugs@lists.postgresql.org
Hi!
I do not know, who supports commitfest site, it why I am writing here.
Every time I add my review in commit fest, I see wrong form -
not all field in form are visible, see attached picture.
As result review generate wrong misleading text, that tests are failed,
like this from my last review:
The following review has been posted through the commitfest application:
make installcheck-world: tested, failed
Implements feature: tested, failed
Spec compliant: tested, failed
Documentation: tested, failed
I tried Safari, Chrome on Mac, Chrome, Edge and Explorer on Windows -
the same result.
The reason: the field "Passed" under field "Tested" is not visible.
I suppose the problem in bootstrap classes. I tried to change them to
documented classes from Bootstrap 3 (as referenced in this page),
and form works.
The solution.
If change the :
<div id="id_review_installcheck" class="form-control"><div>
to
<div id="id_review_installcheck" class="container pull-left"><div>
<div id="id_review_implements" class="form-control"><div>
to
<div id="id_review_implements" class="container pull-left"><div>
<div id="id_review_spec" class="form-control"><div>
to
<div id="id_review_spec" class="container pull-left"><div>
<div id="id_review_doc" class="form-control"><div>
to
<div id="id_review_doc" class="container pull-left"><div>
And all checkbook should be changed from
<input type="checkbox" name="review_installcheck" value="0"
class="form-control" id="id_review_installcheck_0">
to
<input type="checkbox" name="review_installcheck" value="0"
class="form-check-input" id="id_review_installcheck_0">
for better presentation.
In this case all fields will be visible.
--
Best regards,
Vladlen Popolitov.
Attachments:
add_review_flags.png (image/png, 35.7K) download
view image
* Re: Error in form on site commitfest.postgresql.org
2025-02-13 12:46 Error in form on site commitfest.postgresql.org Vladlen Popolitov <v.popolitov@postgrespro.ru>
@ 2025-02-13 14:41 ` Daniel Gustafsson <daniel@yesql.se>
2025-02-21 04:57 ` Re: Error in form on site commitfest.postgresql.org Vladlen Popolitov <v.popolitov@postgrespro.ru>
0 siblings, 1 reply; 4+ messages in thread
From: Daniel Gustafsson @ 2025-02-13 14:41 UTC (permalink / raw)
To: Vladlen Popolitov <v.popolitov@postgrespro.ru>; +Cc: PostgreSQL WWW <pgsql-www@lists.postgresql.org>
I am moving this to pgsql-www@ where the project websites are discussed,
removing pgsql-bugs@ to avoid getting caught in cross-post moderation.
> On 13 Feb 2025, at 13:46, Vladlen Popolitov <v.popolitov@postgrespro.ru> wrote:
>
>
> Hi!
>
> I do not know, who supports commitfest site, it why I am writing here.
>
> Every time I add my review in commit fest, I see wrong form -
> not all field in form are visible, see attached picture.
> As result review generate wrong misleading text, that tests are failed,
> like this from my last review:
> The following review has been posted through the commitfest application:
> make installcheck-world: tested, failed
> Implements feature: tested, failed
> Spec compliant: tested, failed
> Documentation: tested, failed
>
> I tried Safari, Chrome on Mac, Chrome, Edge and Explorer on Windows -
> the same result.
>
> The reason: the field "Passed" under field "Tested" is not visible.
> I suppose the problem in bootstrap classes. I tried to change them to
> documented classes from Bootstrap 3 (as referenced in this page),
> and form works.
>
> The solution.
>
> If change the :
> <div id="id_review_installcheck" class="form-control"><div>
> to
> <div id="id_review_installcheck" class="container pull-left"><div>
>
> <div id="id_review_implements" class="form-control"><div>
> to
> <div id="id_review_implements" class="container pull-left"><div>
>
> <div id="id_review_spec" class="form-control"><div>
> to
> <div id="id_review_spec" class="container pull-left"><div>
>
> <div id="id_review_doc" class="form-control"><div>
> to
> <div id="id_review_doc" class="container pull-left"><div>
>
> And all checkbook should be changed from
> <input type="checkbox" name="review_installcheck" value="0"
> class="form-control" id="id_review_installcheck_0">
> to
> <input type="checkbox" name="review_installcheck" value="0"
> class="form-check-input" id="id_review_installcheck_0">
>
> for better presentation.
>
>
> In this case all fields will be visible.
>
> --
> Best regards,
>
> Vladlen Popolitov.
--
Daniel Gustafsson
Attachments:
add_review_flags.png (image/png, 35.7K) download
view image
* Re: Error in form on site commitfest.postgresql.org
2025-02-13 12:46 Error in form on site commitfest.postgresql.org Vladlen Popolitov <v.popolitov@postgrespro.ru>
2025-02-13 14:41 ` Re: Error in form on site commitfest.postgresql.org Daniel Gustafsson <daniel@yesql.se>
@ 2025-02-21 04:57 ` Vladlen Popolitov <v.popolitov@postgrespro.ru>
2025-02-21 09:23 ` Re: Error in form on site commitfest.postgresql.org Jelte Fennema-Nio <me@jeltef.nl>
0 siblings, 1 reply; 4+ messages in thread
From: Vladlen Popolitov @ 2025-02-21 04:57 UTC (permalink / raw)
To: Daniel Gustafsson <daniel@yesql.se>; +Cc: PostgreSQL WWW <pgsql-www@lists.postgresql.org>
Hi!
I have included the solution into the body of the email.
Should I send some patch to the git of the sites source code?
I did not find any advice to fix www pages except sending
email to this mailing list. I hope it will work later or
ealier.
Daniel Gustafsson писал(а) 2025-02-13 21:41:
> I am moving this to pgsql-www@ where the project websites are
> discussed,
> removing pgsql-bugs@ to avoid getting caught in cross-post moderation.
>
>> On 13 Feb 2025, at 13:46, Vladlen Popolitov
>> <v.popolitov@postgrespro.ru> wrote:
>>
>>
>> Hi!
>>
>> I do not know, who supports commitfest site, it why I am writing here.
>>
>> Every time I add my review in commit fest, I see wrong form -
>> not all field in form are visible, see attached picture.
>> As result review generate wrong misleading text, that tests are
>> failed,
>> like this from my last review:
>> The following review has been posted through the commitfest
>> application:
>> make installcheck-world: tested, failed
>> Implements feature: tested, failed
>> Spec compliant: tested, failed
>> Documentation: tested, failed
>>
>> I tried Safari, Chrome on Mac, Chrome, Edge and Explorer on Windows -
>> the same result.
>>
>> The reason: the field "Passed" under field "Tested" is not visible.
>> I suppose the problem in bootstrap classes. I tried to change them to
>> documented classes from Bootstrap 3 (as referenced in this page),
>> and form works.
>>
The solution.
>>
>> If change the :
>> <div id="id_review_installcheck" class="form-control"><div>
>> to
>> <div id="id_review_installcheck" class="container pull-left"><div>
>>
>> <div id="id_review_implements" class="form-control"><div>
>> to
>> <div id="id_review_implements" class="container pull-left"><div>
>>
>> <div id="id_review_spec" class="form-control"><div>
>> to
>> <div id="id_review_spec" class="container pull-left"><div>
>>
>> <div id="id_review_doc" class="form-control"><div>
>> to
>> <div id="id_review_doc" class="container pull-left"><div>
>>
>> And all checkbook should be changed from
>> <input type="checkbox" name="review_installcheck" value="0"
>> class="form-control" id="id_review_installcheck_0">
>> to
>> <input type="checkbox" name="review_installcheck" value="0"
>> class="form-check-input" id="id_review_installcheck_0">
>>
>> for better presentation.
>>
>>
>> In this case all fields will be visible.
>>
>> --
>> Best regards,
>>
>> Vladlen Popolitov.
>
>
> --
> Daniel Gustafsson
--
Best regards,
Vladlen Popolitov.
^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: Error in form on site commitfest.postgresql.org
2025-02-13 12:46 Error in form on site commitfest.postgresql.org Vladlen Popolitov <v.popolitov@postgrespro.ru>
2025-02-13 14:41 ` Re: Error in form on site commitfest.postgresql.org Daniel Gustafsson <daniel@yesql.se>
2025-02-21 04:57 ` Re: Error in form on site commitfest.postgresql.org Vladlen Popolitov <v.popolitov@postgrespro.ru>
@ 2025-02-21 09:23 ` Jelte Fennema-Nio <me@jeltef.nl>
0 siblings, 0 replies; 4+ messages in thread
From: Jelte Fennema-Nio @ 2025-02-21 09:23 UTC (permalink / raw)
To: Vladlen Popolitov <v.popolitov@postgrespro.ru>; +Cc: Daniel Gustafsson <daniel@yesql.se>; PostgreSQL WWW <pgsql-www@lists.postgresql.org>
On Fri, 21 Feb 2025 at 10:18, Vladlen Popolitov
<v.popolitov@postgrespro.ru> wrote:
> I have included the solution into the body of the email.
> Should I send some patch to the git of the sites source code?
Hi Vladlen,
Thanks for suggesting a fix for this issue! Contributions to the
commitfest app are done through GitHub PRs:
https://github.com/postgres/pgcommitfest
If for some reason that doesn't work for you, sending a patch file to
this thread would also be okay.
Jelte
^ permalink raw reply [nested|flat] 4+ messages in thread
end of thread, other threads:[~2025-02-21 09:23 UTC | newest]
Thread overview: 4+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2025-02-13 12:46 Error in form on site commitfest.postgresql.org Vladlen Popolitov <v.popolitov@postgrespro.ru>
2025-02-13 14:41 ` Daniel Gustafsson <daniel@yesql.se>
2025-02-21 04:57 ` Vladlen Popolitov <v.popolitov@postgrespro.ru>
2025-02-21 09:23 ` Jelte Fennema-Nio <me@jeltef.nl>
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox