Skip to content

Edits to shared-password process #20

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
Merged
ateucher merged 5 commits into from
Jan 15, 2025
Merged

Edits to shared-password process #20

ateucher merged 5 commits into from
Jan 15, 2025

Conversation

@ateucher
Copy link
Contributor

@ateucher ateucher commented Jan 15, 2025 •
edited
Loading

As discussed in coworking:

  • Removed section on using GitHub based access
  • Added sample email to workshop participants
  • Add steps of opening workshop-planning issue and adding to workshop registry
  • Align steps for workshop leads and hub leads, make them overlapping and collaborative
  • Add note about hub capacity (i.e., up to 100 should be fine, over that may require some engineering work by 2i2c)

Deploy preview for Password Access page

@ateucher ateucher requested a review from jules32 January 15, 2025 00:54
@github-actions
Copy link

github-actions bot commented Jan 15, 2025 •
edited
Loading

🚀 Deployed on https://deploy-preview-20--openscapes-cloud-preview.netlify.app

@github-actions github-actions bot temporarily deployed to pull request January 15, 2025 00:54 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 15, 2025 01:05 Inactive
jules32
jules32 reviewed Jan 15, 2025
View reviewed changes
jules32
jules32 approved these changes Jan 15, 2025
View reviewed changes
Copy link
Contributor

@jules32 jules32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Andy! This is great. I made a few edits for clarity but it is good to go! I'll let you merge :)

@ateucher @jules32
Update password-access.qmd
4f92a5b 
Co-authored-by: Julia Stewart Lowndes <julia@openscapes.org>
@github-actions github-actions bot temporarily deployed to pull request January 15, 2025 20:34 Inactive
@ateucher @jules32
Apply suggestions from code review
bfb5a48 
Co-authored-by: Julia Stewart Lowndes <julia@openscapes.org>
@ateucher
Copy link
Contributor Author

ateucher commented Jan 15, 2025

Excellent suggestions, thanks so much @jules32!

@github-actions github-actions bot temporarily deployed to pull request January 15, 2025 20:35 Inactive
@ateucher ateucher merged commit 620a8f4 into main Jan 15, 2025
2 checks passed
@ateucher ateucher deleted the shared-password-admin branch January 15, 2025 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jules32 jules32 jules32 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ateucher @jules32