Skip to content

feat: add basic syncing from lim_medlemsinfo db#2174

Draft
BragonSB wants to merge 1 commit into
masterfrom
brage/web-94-sync-info-from-lim_medlemsinfo
Draft

feat: add basic syncing from lim_medlemsinfo db#2174
BragonSB wants to merge 1 commit into
masterfrom
brage/web-94-sync-info-from-lim_medlemsinfo

Conversation

@BragonSB

@BragonSB BragonSB commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Closes: WEB-94

Not done

Still kinda a bit unsure about what we should and shouldn't prioritize when syncing

@BragonSB BragonSB self-assigned this Jun 8, 2026
return None
if (user.mdb_last_updated is not None and (timezone.now() - user.mdb_last_updated).total_seconds() < minute_threshold * 60):
return None

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gjerne kjør en sammenligning av verdier først her for å se om vi i det hele tatt trenger å oppdatere

@BragonSB BragonSB Jun 29, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

På linje 50 det? Kan være jeg misforstår her, men for å sammenlikne så må jeg vel uansett gjør et SQL call? Hvis du mener at jeg burde legge til en sammenlikning etter SELECT callet, men før selve oppdateringen på linje 59-> så er jeg enig

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mente etter SELECTen ja

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants