Registry name tuples #23

jeffgrunewald · 2021-08-11T05:30:22Z

Addresses the majority of concerns raised in #21 with respect to:

Better defining the API for locating processes that need to have messages sent to them
Converts potentially unsafe atom interpolated process names to via-tuples with string values
Exposes the new API as a helper function at the top level of the library to easily generate the via tuple needed to locate/message a process, setting the default to the most commonly needed values.

lib/stargate.ex

BrendanBall · 2021-08-11T13:40:39Z

Thanks for doing the work 🥇
I've tested it in our project except for using Stargate.registry_key because of the above and it seems to work well.

jeffgrunewald added 5 commits July 29, 2021 01:44

change dynamic process names to via tuples

50de557

Adding persistence to process name via tuples

ceacaf8

adding registry key builder function

2509e96

formatting, credo, and use registry_key func

d87dad7

updating README

4ab343a

jeffgrunewald self-assigned this Aug 11, 2021

BrendanBall reviewed Aug 11, 2021

View reviewed changes

lib/stargate.ex Outdated Show resolved Hide resolved

jeffgrunewald added 2 commits August 12, 2021 23:31

allow overriding registry_key function registry name

5c449af

doc updates for registry_key function opt

ccb195f

jeffgrunewald merged commit d80e537 into main Aug 13, 2021

jeffgrunewald deleted the registry_name_tuples branch August 13, 2021 04:24

jeffgrunewald mentioned this pull request Aug 13, 2021

problem with registry keys #21

Closed

Provide feedback