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

#437 - Added documentation for specify and no-ring 0 specify #438

Merged
merged 8 commits into from
Jan 26, 2025

Conversation

Mr-KayZ
Copy link
Contributor

@Mr-KayZ Mr-KayZ commented Jan 25, 2025

LMK if this is works!

@Mr-KayZ Mr-KayZ linked an issue Jan 25, 2025 that may be closed by this pull request
Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-mushroom-0564bdf0f-438.eastus2.5.azurestaticapps.net

@TheKrol
Copy link
Collaborator

TheKrol commented Jan 25, 2025

Need to update the no-ring download to the correct one. I'm not sure if there is a link for it right now, just might need to get it off github.
Found the link: https://github.com/Spec-ify/specify/releases/latest/download/Specify_noring0.exe

Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-mushroom-0564bdf0f-438.eastus2.5.azurestaticapps.net

@Mr-KayZ
Copy link
Contributor Author

Mr-KayZ commented Jan 25, 2025

Added the following changes:

  • Changed specify section in Making a System Dossier - Added links for both Specify articles (normal and no ring 0)
  • Changed up wording of both specify and no ring 0 articles with active feedback (thanks again xscissors, Krol and Pipe!)

Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-mushroom-0564bdf0f-438.eastus2.5.azurestaticapps.net

Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-mushroom-0564bdf0f-438.eastus2.5.azurestaticapps.net

Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-mushroom-0564bdf0f-438.eastus2.5.azurestaticapps.net

@PipeItToDevNull
Copy link
Member

Combining the callouts is the last thing I see, maybe like below

No sensitive information is included in the report, we will not ask you to share private information. You can vew an example specify report here.

More information can be found on our README on GitHub.

And I don't think we need the ! on callouts https://rtech.support/meta/contributing/#alerts

@Mr-KayZ
Copy link
Contributor Author

Mr-KayZ commented Jan 26, 2025

And I don't think we need the ! on callouts https://rtech.support/meta/contributing/#alerts

Nope we definitely need those:
image

As for the rest, done for both no-ring and default

Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-mushroom-0564bdf0f-438.eastus2.5.azurestaticapps.net

@PipeItToDevNull
Copy link
Member

And I don't think we need the ! on callouts https://rtech.support/meta/contributing/#alerts

Nope we definitely need those:

image

As for the rest, done for both no-ring and default

I meant on the end "Important!"

Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-mushroom-0564bdf0f-438.eastus2.5.azurestaticapps.net

@PipeItToDevNull
Copy link
Member

I fixed that last thing so should be good to go

@PipeItToDevNull PipeItToDevNull merged commit 5cc31b7 into master Jan 26, 2025
3 checks passed
@PipeItToDevNull PipeItToDevNull deleted the 437-specify-factoids branch January 26, 2025 17:05
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.

Specify factoids
3 participants