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.

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

Available Slots Compatibility #512

Merged
merged 1 commit into from
Jun 19, 2022
Merged

Conversation

TurtIeSocks
Copy link
Collaborator

  • Updates SQL query to use a dynamic column name based on a startup check
  • Maintains backwards compatibility for older RDM clients that are still using availble_slots

- Updates SQL query to use a dynamic column name based on a startup check
- Maintains backwards compatibility for older RDM clients that are still using availble_slots
@TurtIeSocks TurtIeSocks merged commit 92baddf into develop Jun 19, 2022
@TurtIeSocks TurtIeSocks deleted the available-slots-adjustments branch June 19, 2022 02:44
@Fabio1988
Copy link
Contributor

This misspelling is fixed in rdm master since december.... I kept old column for compatibility.

Why not simple use it and remove the magic to keep it clean? Someone who runs RDM master from december is save. Someone who's not running this state can also run RM from an older state 🤣

@TurtIeSocks
Copy link
Collaborator Author

More backwards compatibility = less dumb questions to answer later 😄

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