Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor or remove properties #47

Closed
jonthegeek opened this issue Sep 13, 2023 · 1 comment
Closed

Refactor or remove properties #47

jonthegeek opened this issue Sep 13, 2023 · 1 comment
Milestone
0.1: through Auth

Comments

@jonthegeek
Copy link
Owner

The properties have a hack for error reporting. Refactor them to get rid of that hack. Maybe with validators? Maybe by converting them to classes?

I still want to use stbl for coersion, but find a better way to do it. A class constructor is probably a friendlier place.
@jonthegeek jonthegeek added this to the 0.1 milestone Sep 13, 2023
@jonthegeek
Copy link
Owner Author

After digging into this more... I think I'm doing it as close to the "right" way as possible right now. Might be best to just capture stbl errors and let S7 handle them?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
{rapid} releases
Archived in project
Milestone
0.1: through Auth
Development

No branches or pull requests
1 participant
@jonthegeek