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

docs: (IAC-1462) remove references to multi-tenancy documentation #552

Merged
merged 1 commit into from
May 29, 2024

Conversation

riragh
Copy link
Member

@riragh riragh commented May 20, 2024

Changes:

SAS Viya currently has pause the support for Multi-tenancy, so all the references to Multi-tenancy documentation have been removed.

@riragh riragh added the documentation Improvements or additions to documentation label May 20, 2024
@riragh riragh requested review from dhoucgitter, thpang and jarpat May 20, 2024 16:05
@riragh riragh self-assigned this May 20, 2024
@jarpat
Copy link
Contributor

jarpat commented May 20, 2024

@riragh should the example file be removed as well?

@dhoucgitter
Copy link
Member

@riragh should the example file be removed as well?

good point, not sure why we'd want to keep that if deleting the doc files.

@riragh
Copy link
Member Author

riragh commented May 20, 2024

@jarpat, @dhoucgitter So the idea is to keep the code in as is, just remove the doc references. So this would be like undocumented feature. I am not sure if example file would be considered as doc or code. If we you all think we should remove I am good to remove. Please let me know.

@dhoucgitter
Copy link
Member

@jarpat, @dhoucgitter So the idea is to keep the code in as is, just remove the doc references. So this would be like undocumented feature. I am not sure if example file would be considered as doc or code. If we you all think we should remove I am good to remove. Please let me know.

I suspect that if our intent is that the MT behavior/code should still be in place to support using Multi-Tenancy, just not document how to do that, then removing the example file is probably not a good idea.

@jarpat jarpat merged commit be68988 into main May 29, 2024
3 checks passed
@jarpat jarpat deleted the IAC-1462 branch May 29, 2024 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants