这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@blu25
Copy link
Collaborator

@blu25 blu25 commented Jan 18, 2024

This updates the line that clears out automatic beacon data whose once=True to use the new automatic beacon data map that was added in #133.

One concern I have about this is whether or not the get the automatic beacon data mapping to use algorithm is building a map of references to or copies of the original mapping. If it's the latter, then we'll need to do further refactoring to make sure that the things in the various mappings can be modified by the "maybe send automatic beacon" algorithm.


Preview | Diff

@blu25 blu25 marked this pull request as ready for review January 18, 2024 23:33
@blu25 blu25 requested a review from domfarolino January 18, 2024 23:33
@domfarolino domfarolino merged commit d6a5a54 into master Jan 19, 2024
@domfarolino domfarolino deleted the liam-fix-beacon-data-map branch January 19, 2024 15:49
github-actions bot added a commit that referenced this pull request Jan 19, 2024
SHA: d6a5a54
Reason: push, by domfarolino

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants