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

Only show create RFD button to internal users #64

Open
mattdailis opened this issue Aug 31, 2024 · 4 comments
Open

Only show create RFD button to internal users #64

mattdailis opened this issue Aug 31, 2024 · 4 comments

Comments

@mattdailis
Copy link

Screenshot 2024-08-30 at 7 22 33 PM

new.sh is in the private rfd repo. I imagine it's relatively simple, and not proprietary, so it would be great if it could be made public!

Of course, it's convenient for it to be colocated with the rfd repo, so I don't think I'm proposing it be moved somewhere else. Perhaps an empty rfd-template repo for others to use as a starting point would be appropriate?

@david-crespo
Copy link
Contributor

david-crespo commented Aug 31, 2024

I think we just want to make this line conditional on some attribute of the user that indicates they’re internal.

<NewRfdButton />

I’m sure we can share the script and template, but it’s not very interesting: basically it creates a directory named 0123 containing a placeholder RFD asciidoc file (with the author and provided title templated in) in a branch also named 0123, and it pushes the branch.

Here is the template:

:showtitle:
:toc: left
:numbered:
:icons: font
:state: prediscussion
:discussion:
:revremark: State: {state} | {discussion}
:authors: Andy Hertzfeld <[email protected]>; Neal Firth <[email protected]>

= RFD <Number> <Title>
{authors}

== Determinations

== Open Questions

== Security Considerations

[bibliography]
== External References

@mattdailis
Copy link
Author

Thanks! That's enough to get me started.

In case anyone else finds it useful to use as a starting point, here's my crude first cut based on David's description above: https://github.com/mattdailis/rfd-template

@david-crespo david-crespo changed the title Private New RFD command referenced in public site Only show create RFD button to internal users Aug 31, 2024
@augustuswm
Copy link
Contributor

augustuswm commented Aug 31, 2024

Agreed, we should limit that button to only users that have permission to create a new RFD. I have to get a new release of the api deployed but we have a permission variant now that we should be able to check to verify that.

Related to the new rfd script, we don't have any plains to make that not work, but we are finishing some work on api endpoints for creating a new RFD that does the same mechanical bits on behalf of the user. It is likely that we will update the create RFD button to show both options to the user.

@benjaminleonard
Copy link
Collaborator

FWIW I'm working on a PR to create a new RFD from that modal directly without the script. Based on this PR.

image

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

No branches or pull requests

4 participants