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

Adding DRAFT status to DC files #37

Merged
merged 10 commits into from
Sep 21, 2021
Merged

Conversation

fsundermeyer
Copy link
Contributor

No description provided.

- fix content
- remove entry from list of related topics
- add sentence to summary
Copy link
Contributor

@taroth21 taroth21 left a comment

Choose a reason for hiding this comment

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

IIRC, the e-mail-address for bugs needs to be the one registered/used within Bugzilla, therefore changing the one for Dima.

<dm:product>Product Name</dm:product>
<dm:assignee>assignee@suse.com</dm:assignee>
<dm:product>PUBLIC SUSE Linux Enterprise Server 15 SP3</dm:product>
<dm:assignee>dpopov@suse.com</dm:assignee>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<dm:assignee>dpopov@suse.com</dm:assignee>
<dm:assignee>dmitri.popov@suse.com</dm:assignee>

<dm:product>Product Name</dm:product>
<dm:assignee>assignee@suse.com</dm:assignee>
<dm:product>PUBLIC SUSE Linux Enterprise Server 15 SP3</dm:product>
<dm:assignee>dpopov@suse.com</dm:assignee>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<dm:assignee>dpopov@suse.com</dm:assignee>
<dm:assignee>dmitri.popov@suse.com</dm:assignee>

<dm:product>Product Name</dm:product>
<dm:assignee>assignee@suse.com</dm:assignee>
<dm:product>PUBLIC SUSE Linux Enterprise Server 15 SP3</dm:product>
<dm:assignee>dpopov@suse.com</dm:assignee>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<dm:assignee>dpopov@suse.com</dm:assignee>
<dm:assignee>dmitri.popov@suse.com</dm:assignee>

<dm:product>Product Name</dm:product>
<dm:assignee>assignee@suse.com</dm:assignee>
<dm:product>PUBLIC SUSE Linux Enterprise Server 15 SP3</dm:product>
<dm:assignee>dpopov@suse.com</dm:assignee>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<dm:assignee>dpopov@suse.com</dm:assignee>
<dm:assignee>dmitri.popov@suse.com</dm:assignee>

@@ -19,7 +19,7 @@
<dm:bugtracker>
<dm:url>https://bugzilla.suse.com/enter_bug.cgi</dm:url>
<dm:component>Documentation</dm:component>
<dm:product>Product Name</dm:product>
<dm:product>PUBLIC SUSE Linux Enterprise Server 15 SP3</dm:product>
<dm:assignee>[email protected]</dm:assignee>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<dm:assignee>dpopov@suse.com</dm:assignee>
<dm:assignee>dmitri.popov@suse.com</dm:assignee>

@fsundermeyer fsundermeyer force-pushed the draft_status_bugzilla_link branch from a887e56 to a05f12e Compare September 21, 2021 07:40
@fsundermeyer fsundermeyer merged commit b00b3fd into main Sep 21, 2021
@fsundermeyer fsundermeyer deleted the draft_status_bugzilla_link branch September 21, 2021 08:27
@ghost
Copy link

ghost commented Oct 4, 2021

In a way, this is fun -- I specifically updated Docserv2 to not add --draft to the command line of docs marked as beta just days earlier, because --draft also triggers --meta + --remarks as well, and both of those imo look a bit unprofessional on a published website.

So,

  1. Sorry for creating this sort of busywork for you.
  2. Should we do something about the fact that DAPS includes meta information and remarks in every --draft build? Or do think that is still expected?

(@fsundermeyer / @taroth21 )

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.

2 participants