chore(document/templates): remove all references#192
Conversation
|
If the ui isn't using them, this seems like a fine step in deprecating documents, otherwise just move the documents stuff out of |
|
Cheers @glinton |
|
@GeorgeMac re: making sure the UI doesn't use these paths, I created a branch from this one an re-based it with master so that I could point the UI at it and try out a build. The UI builds fine with this without any typescript errors so I think we can be quite confident that the UI does not use these paths. Would there be anything else holding up merging this PR? It'd be nice to get this sorted out since OSS already does not have most of these routes and we should deprecate the last remaining read-only route there for consistency. |
|
@wbaker85 that is amazing. Thank you for doing that 🙏 I'm happy to go ahead and merge in that case. |
3 participants
Closes #161
Personally, I think it is time. But, I also understand this is a breaking change.
So I open this to get that conversation rolling again.
I found from our cloud logs the only activity we had was from our post deployment tests.
So I am not worried about cloud (and I think I removed all the code from OSS already 🤔 - if my memory serves).