You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@weex.apache.org by kuldeepkeshwar <gi...@git.apache.org> on 2018/06/28 05:35:35 UTC

[GitHub] incubator-weex pull request #1300: Added support for multi-value headers(WXS...

GitHub user kuldeepkeshwar opened a pull request:

    https://github.com/apache/incubator-weex/pull/1300

    Added support for multi-value headers(WXStreamModule)

    First of all, thank you for your contribution! 
    
    Please makes sure that these checkboxes are checked before submitting your PR, thank you!
    
    <!-- 
    How to use github markdown?
    ### checkable example
    - [x] checked
    - [ ] not checked
    More github Markdown info to see https://guides.github.com/features/mastering-markdown/
    -->
    
    CheckList:
    
    **if** *isBugFix* **:**
    
      * [ ] Make sure that you add at least one unit test for the bug which you had fixed.
    
    **elif** *isNewFeature* **:**
    
      * [ ] Update API docs for the component.
      * [ ] Update/Add demo to demonstrate new feature.
      * [ ] Update test scripts for the feature.
      * [ ] Add unit tests for the feature.
    
    <!--
    (请在***提交***前删除这段描述)
    
    Notes: Weex will move into Apache Software Foundation (ASF) on Feb 24 2017.
    
    Our new GitHub repo is https://github.com/apache/incubator-weex
    
    After Feb 24 2017, we only accept pull requests from https://github.com/apache/incubator-weex
    
    Thank you for your support.
    
    ----
    
    注意:Weex 将于 2017-02-24 迁移至 Apache 基金会
    
    届时我们会使用新的 GitHub 仓库:https://github.com/apache/incubator-weex 并在那里继续接受大家的 pull request。
    
    更多详情请关注:https://github.com/weexteam/article/issues/130
    
    感谢理解和支持
    
    -->
    
    <!--
    (请在***提交***前删除这段描述)
    It's ***RECOMMENDED*** to submit typo fix, new demo, tiny bugfix and large feature to `master` branch.
        
    See [Branch Strategy](https://github.com/alibaba/weex/blob/dev/CONTRIBUTING.md#branch-management) for more detail.
    
    ----
    
    错别字修改、新 demo、较小的 bugfix、甚至较大的功能都可以直接提到 `master` 分支;
    
    查看完整的[分支策略 (英文)](https://github.com/alibaba/weex/blob/dev/CONTRIBUTING.md#branch-management)。
    
    -->


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/kuldeepkeshwar/incubator-weex master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-weex/pull/1300.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1300
    
----
commit 20bef5c22a1248ec9ec9906d008ef712a1bcf112
Author: anotherjsguy <ku...@...>
Date:   2018-06-28T02:58:33Z

    Merge pull request #1 from apache/master
    
    synced repo

commit 0a59d9d1fefec612ede32d6ed10ad6e242291a27
Author: kuldeepkeshwar <ku...@...>
Date:   2018-06-28T03:22:39Z

    added handling for multi value headers

commit 0afdfbe7250d4c587d7b182a488c55ab70f616bf
Author: kuldeepkeshwar <ku...@...>
Date:   2018-06-28T05:29:16Z

    Revert "added handling for multi value headers"
    
    This reverts commit 0a59d9d1fefec612ede32d6ed10ad6e242291a27.

commit 7358311dac6551ac62bf506185d1188e04eae022
Author: kuldeepkeshwar <ku...@...>
Date:   2018-06-28T05:33:22Z

    * [android] fix #WEEX-482, support for multValue headers(WXStreamModule)

----


---

[GitHub] incubator-weex issue #1300: Added support for multi-value headers(WXStreamMo...

Posted by weex-bot <gi...@git.apache.org>.
Github user weex-bot commented on the issue:

    https://github.com/apache/incubator-weex/pull/1300
  
    
    <!--
      0 failure: 
      0 warning: 
      2 messages
      
    -->
    
    
    
    <table>
      <thead>
        <tr>
          <th width="50"></th>
          <th width="100%" data-danger-table="true">Messages</th>
        </tr>
      </thead>
      <tbody><tr>
          <td>:book:</td>
          <td>has no jsfm file changed.</td>
        </tr>
      
    <tr>
          <td>:book:</td>
          <td>jsfm test finished.</td>
        </tr>
      </tbody>
    </table>
    
    
    <p align="right">
      Generated by :no_entry_sign: <a href="http://github.com/danger/danger-js/">dangerJS</a>
    </p>



---

[GitHub] incubator-weex issue #1300: [WEEX-482][Android] Added support for multi-valu...

Posted by gubaojian <gi...@git.apache.org>.
Github user gubaojian commented on the issue:

    https://github.com/apache/incubator-weex/pull/1300
  
    use git rebase to merge your commit multi history to one.


---