Skip to content

Add Pull Request builds page to settings #206

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

Merged
merged 9 commits into from
Jul 1, 2024
Merged

Conversation

ericholscher
Copy link
Member

@ericholscher ericholscher commented Aug 22, 2023

This is a basic form for now,
but we can spruce it up before it goes live.

Depends on readthedocs/readthedocs.org#10656

This is a basic form for now,
but we can spruce it up before it goes live.
@ericholscher ericholscher requested a review from a team as a code owner August 22, 2023 20:49
@ericholscher ericholscher requested a review from agjohnson August 22, 2023 20:49
Copy link
Contributor

@agjohnson agjohnson left a comment

Choose a reason for hiding this comment

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

Just some copy changes, looks good!

@ericholscher
Copy link
Member Author

@agjohnson Thanks for your review 🎉. I don't think I'll push this any further this week before I head out, so feel free to merge it if you have time to clean things up for the beta 👍

@agjohnson
Copy link
Contributor

Sounds good, I'll merge this when the underlying PR goes in. I can help this out if it needs anything.

@agjohnson agjohnson added the Status: blocked Issue is blocked on another issue label Aug 24, 2023
@ericholscher ericholscher self-assigned this Jan 31, 2024
@agjohnson agjohnson added this to the New dashboard features milestone Mar 28, 2024
@ericholscher ericholscher requested a review from agjohnson June 26, 2024 18:02
{% csrf_token %}
{{ form | crispy }}

<input class="ui {% if form.errors and form.is_disabled %}disabled{% endif %} primary button" type="submit"
Copy link
Contributor

Choose a reason for hiding this comment

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

This works okay for now, I've noted to undo this pattern with readthedocs/readthedocs.org#11270

@ericholscher ericholscher merged commit d740ffa into main Jul 1, 2024
@ericholscher ericholscher deleted the pull-request-settings branch July 1, 2024 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: blocked Issue is blocked on another issue
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Project: add project admin dashboard for pull request builds
2 participants