djangogirls icon indicating copy to clipboard operation
djangogirls copied to clipboard

New coaches "saved" but seem to be auto-deleting

Open mUtterberg opened this issue 5 years ago • 9 comments

screen shot 2018-08-28 at 5 32 47 pm screen shot 2018-08-28 at 5 32 59 pm

I've tried to add a coach twice today. Each time, I get a "saved" confirmation, but the coaches section in my admin page shows no coaches and throws me a "no coach with ID ####" error.

mUtterberg avatar Aug 28 '18 21:08 mUtterberg

@mUtterberg what page is this happening? Also can you provide a step by step of how to reproduce it?

ramonsaraiva avatar Oct 03 '18 22:10 ramonsaraiva

I'm unable to save coaches from the admin tab for viewing coaches, but I was able to add them as page dependencies in the "edit website contents" tab for web content pages in admin. Will comment in more detail tomorrow, but I did find a workaround.

--Marissa Utterberg Data Science & Web Development Find Utterberg Data & Development at utterbergdatadev.com http://utterbergdatadev.com

On Wed, Oct 3, 2018 at 6:25 PM Ramon Saraiva [email protected] wrote:

@mUtterberg https://github.com/mUtterberg what page is this happening? Also can you provide a step by step of how to reproduce it?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/DjangoGirls/djangogirls/issues/484#issuecomment-426824111, or mute the thread https://github.com/notifications/unsubscribe-auth/Ahi3_AaiWmTV4FpWd_nUmjSoPBaTEU8Bks5uhTlNgaJpZM4WQcCl .

mUtterberg avatar Oct 03 '18 22:10 mUtterberg

Thank you!

ramonsaraiva avatar Oct 03 '18 23:10 ramonsaraiva

I've just hit this issue too. I think the problem is in the CoachAdmin.get_queryset method: https://github.com/DjangoGirls/djangogirls/blob/master/coach/admin.py#L21

It's restricting you to viewing/editing coaches that are already linked from an EventPageContent object for an event for which the user is part of the team, hence the (erroneous) "does not exist" message.

A possible solution would be require adding the coach to an EventPageContent using an admin inline on the creation form. If that sounds like a good idea then I'm happy to create a PR.

bcdickinson avatar Aug 30 '19 16:08 bcdickinson

I've just hit this issue too. I think the problem is in the CoachAdmin.get_queryset method: https://github.com/DjangoGirls/djangogirls/blob/master/coach/admin.py#L21

Permalink for posterity: https://github.com/DjangoGirls/djangogirls/blob/3b4a234ac1c92fffb83b551cce5da81ee3d83692/coach/admin.py#L21

das-g avatar Aug 30 '19 17:08 das-g

This issue seems to be still open. I am currently organizing an event and I am running into the same problem.

If someone would drop a workaround as the pull requests don't seem to get approved that would be great.

@mUtterberg it's been a while ago now, but did you manage to get a better workaround?

Otherwise I will see during the next weeks if I can build a little html-boilerplate with inline css or something that can be copy pasted into the admin view form that looks good as a workaround. It would totally force us to add every coach manually in HTML, but it might be the only way for now.

I will post the code here once I am done. This might get others running into this issue a little head start when spinning up their event website.

DataCerealz avatar May 03 '21 13:05 DataCerealz

I'll check this.

vanessa avatar Sep 01 '21 03:09 vanessa

Perhaps this was fixed in https://github.com/DjangoGirls/djangogirls/pull/750?

vanessa avatar Dec 26 '22 11:12 vanessa

There's a possibility that this is related to #956

vanessa avatar Apr 30 '24 22:04 vanessa